oxidecomputer / oxidecomputer/omicron
Enable actual timeseries schema updates
Open
@bnaecker is already working on this.
Since Jun 25, 2024.
Metrics
oximeter
- Dominant language
- Rust
- Stars
- 572
- Forks
- 97
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 96
Description
As part of #5889, we added a way to define timeseries in TOML files to make it possible to update timeseries schema. Right now, there are tests around the logic of handling those updates, but there is a check in non-test builds that precludes defining any version other than 1. That lets us build the tooling for making updates, while not actually allowing timeseries to be updated until all the pieces are in place. This issue tracks removing that, letting folks actually update timeseries schema.
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.
Assessment
This issue has not been assessed yet.