oxidecomputer / oxidecomputer/omicron
remove support for old-format services ledger and RSS service plan
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 572
- Forks
- 97
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 96
Description
In #4466 we're changing the format of the sled agent services ledger and the RSS service plan. There's code (and tests) to support migration for existing systems. Once all deployments that we care about are running bits with #4466 in them (which looks like it'll be release 5 or 6), we should consider removing this migration code and the associated tests. As of now, the deployments we care about are probably the customer systems, plus dogfood and rack3, and I expect they'd all be updated in the next month or two.
Originally posted by @smklein in https://github.com/oxidecomputer/omicron/pull/4466#discussion_r1409760976
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the migration code and associated tests for the sled agent services ledger and RSS service plan format change described in #4466. Confirm the relevant deployments have adopted the new format, then remove the migration support and its tests; done means existing deployments no longer need the old-format path.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100