Skip to content

T7994: fix regression in check for previous installations on install - #4849

Merged
jestabro merged 1 commit into
vyos:currentfrom
jestabro:fix-check-for-previous-installations
Nov 17, 2025
Merged

T7994: fix regression in check for previous installations on install#4849
jestabro merged 1 commit into
vyos:currentfrom
jestabro:fix-check-for-previous-installations

Conversation

@jestabro

Copy link
Copy Markdown
Contributor

Change summary

After T7836: move bind mount of /config to vyos-1x, the bind mount is configured at boot, not within the initrd. On image install, one needs to check for previous installations in the resident directory /opt/vyatta/etc/config of the mounted disk.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

How to test / Smoketest result

As in the task description, one can mount the built image in an existing vm and boot from iso; image install. One will see previous image found and config dir copied.

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

After T7836: move bind mount of /config to vyos-1x, the bind mount is
configured at boot, not within the initrd. On image install, one needs
to check for previous installations in the resident directory
/opt/vyatta/etc/config of the mounted disk.
@jestabro jestabro self-assigned this Nov 14, 2025
@github-actions

Copy link
Copy Markdown

👍
No issues in PR Title / Commit Title

@github-actions

Copy link
Copy Markdown

CI integration 👍 passed!

Details

CI logs

  • CLI Smoketests (no interfaces) 👍 passed
  • CLI Smoketests VPP 👍 passed
  • CLI Smoketests (interfaces only) 👍 passed
  • Config tests 👍 passed
  • Config tests VPP 👍 passed
  • RAID1 tests 👍 passed
  • TPM tests 👍 passed

@alexk37

alexk37 commented Nov 16, 2025

Copy link
Copy Markdown
Contributor

Fix works for me, previous installation found and config copied

@sever-sever sever-sever left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked by @alexk37

@sarthurdev sarthurdev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change makes sense and fix verified by Alex.

@jestabro jestabro added the bp/circinus Create automatic backport for circinus label Nov 17, 2025
@jestabro
jestabro merged commit 2a9b37a into vyos:current Nov 17, 2025
18 of 19 checks passed
@vyosbot vyosbot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Nov 17, 2025
@jestabro
jestabro deleted the fix-check-for-previous-installations branch December 5, 2025 16:30
@andamasov andamasov removed the bp/circinus Create automatic backport for circinus label May 9, 2026
@github-actions github-actions Bot added the rebase label May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

6 participants