Skip to content

reconfigurator-cli could let you set SP versions #8273

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

Merged
merged 52 commits into from
Jun 13, 2025

Conversation

davepacheco
Copy link
Collaborator

@davepacheco davepacheco commented Jun 4, 2025

Depends on #8269.

This PR:

  • augments reconfigurator-cli's internal model of sleds to include cabooses for the SP's active and inactive slots
    • for simulated models: fills in arbitrary defaults
    • for models loaded from a saved file: loads data out of the saved inventory
  • shows the sled serial number and these new versions in the output of sled-show
  • includes these cabooses in inventory collections generated by inventory-generate
  • adds a command sled-update-sp for setting the version in these caboose fields
  • adds tests for all this

The goal here is to work toward being able to demo/test the planner behavior around pending MGS updates. With this PR (and #8269, on which it's based), I was able to:

  • generate a blueprint with a configured MGS update
  • demo behavior of seeing that an update has become impossible:
    • use sled-update-sp to set the inactive slot to something else, invalidating the configured update
    • run the planner and see it remove the configured update because it's impossible. (It doesn't create a replacement yet because this requires teaching reconfigurator-cli about TUF artifacts, which it doesn't know about yet.)
  • demo behavior of seeing that an update has finished:
    • use sled-update-sp to set the active slot to the version we're trying to update to
    • run the planner and see it remove the configured update because it's done

There are some limitations here:

@davepacheco davepacheco self-assigned this Jun 4, 2025
Copy link
Contributor

@karencfv karencfv left a comment

Choose a reason for hiding this comment

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

Looks great! Just a small optional nit

Base automatically changed from dap/sp-planning-real to main June 13, 2025 19:27
@davepacheco davepacheco merged commit 8a92e14 into main Jun 13, 2025
17 checks passed
@davepacheco davepacheco deleted the dap/reconfigurator-cli-set-sp-versions branch June 13, 2025 20:50
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