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

remove old, pre-reconfigurator upgrade APIs #7431

Merged
merged 1 commit into from
Jan 29, 2025
Merged

Conversation

iliana
Copy link
Contributor

@iliana iliana commented Jan 29, 2025

This removes:

  • the Sled Agent API update_artifact, which was intended for Nexus to command Sled Agent to apply an arbitrary update, immediately, whatever that meant for the individual artifact.
  • the Nexus internal API cpapi_artifact_download, which was intended for Sled Agent to be able to download an artifact from a local cache managed by Nexus and expected to be long-lived.
  • the test which tested update_artifact.

The replacements for this functionality will be:

  • the TUF Repo Depot, in which Nexus immediately pushes artifacts onto sleds so that it doesn't have to maintain persistent storage.
  • Reconfigurator, which will plan and execute updates in a coordinated fashion across the rack.
  • New/existing endpoints specific to artifact kinds for applying updates; for example for zones, Nexus will put a new zone configuration to Sled Agent that tells it to use a specific zone image from the Repo Depot.

I spotted that this cleanup is needed while working on #4411.

@iliana iliana enabled auto-merge (squash) January 29, 2025 20:57
@iliana iliana merged commit 1774d5d into main Jan 29, 2025
17 checks passed
@iliana iliana deleted the iliana/tuf-cleanup branch January 29, 2025 21:49
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.

2 participants