Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change boot order #876

Merged
merged 4 commits into from
Dec 19, 2024
Merged

Change boot order #876

merged 4 commits into from
Dec 19, 2024

Conversation

mattiaswal
Copy link
Contributor

@mattiaswal mattiaswal commented Dec 17, 2024

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

@mattiaswal mattiaswal marked this pull request as draft December 17, 2024 14:16
@mattiaswal mattiaswal marked this pull request as ready for review December 17, 2024 20:41
@mattiaswal mattiaswal force-pushed the change-boot-order branch 4 times, most recently from f110468 to be8acae Compare December 18, 2024 05:47
@mattiaswal mattiaswal requested review from wkz and troglobit December 18, 2024 07:30
@mattiaswal mattiaswal force-pushed the change-boot-order branch 2 times, most recently from 793f085 to 9596bc1 Compare December 18, 2024 12:40
Copy link
Contributor

@troglobit troglobit left a comment

Choose a reason for hiding this comment

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

Like wkz said, it's the single most important feature of the operating system, so I'd like to tip my hat and say: great work on this! 😮 🎉

I only have a few minor grammar and stylistic comments.

Also, very cool to learn about the YANG feature ordered-by user! 🤓

src/confd/src/infix-system-software.c Outdated Show resolved Hide resolved
src/confd/src/infix-system-software.c Outdated Show resolved Hide resolved
src/confd/src/infix-system-software.c Outdated Show resolved Hide resolved
src/confd/src/infix-system-software.c Outdated Show resolved Hide resolved
src/confd/src/infix-system-software.c Outdated Show resolved Hide resolved
src/confd/src/infix-system-software.c Outdated Show resolved Hide resolved
@mattiaswal mattiaswal force-pushed the change-boot-order branch 2 times, most recently from 18146a7 to 3b20a29 Compare December 18, 2024 16:42
@mattiaswal mattiaswal requested a review from troglobit December 18, 2024 16:42
Copy link
Contributor

@troglobit troglobit left a comment

Choose a reason for hiding this comment

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

👍

@mattiaswal mattiaswal force-pushed the change-boot-order branch 4 times, most recently from 2d78c94 to 092c82d Compare December 19, 2024 08:43
Copy link
Contributor

@wkz wkz left a comment

Choose a reason for hiding this comment

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

Doin' the lord's work! 🙏

src/confd/yang/infix-system-software.yang Show resolved Hide resolved
test/infamy/tap.py Outdated Show resolved Hide resolved
test/case/ietf_system/upgrade/test.py Outdated Show resolved Hide resolved
test/infamy/restconf.py Show resolved Hide resolved
If a test need to clean up itself disregarding test status,
it can add an optional test-cleanup phase.

This is useful if the test in some circomstances leave the device in a
bad state. This can be used to restore the unit to its ordinary state.
* Use new cleanup in infamy to ensure the boot order gets restored
* Use new boot order in operational and use RPC to set boot order
  to remove all SSH commands.
Copy link
Contributor

@wkz wkz left a comment

Choose a reason for hiding this comment

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

:shipit:

@mattiaswal mattiaswal merged commit a8545e3 into main Dec 19, 2024
6 checks passed
@mattiaswal mattiaswal deleted the change-boot-order branch December 19, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants