Skip to content

Use latest API version in Oximeter server#10445

Merged
bnaecker merged 1 commit into
mainfrom
ben/use-latest-oximeter-version
May 14, 2026
Merged

Use latest API version in Oximeter server#10445
bnaecker merged 1 commit into
mainfrom
ben/use-latest-oximeter-version

Conversation

@bnaecker
Copy link
Copy Markdown
Collaborator

No description provided.

@bnaecker bnaecker requested a review from sunshowers May 14, 2026 16:13
@bnaecker
Copy link
Copy Markdown
Collaborator Author

I've been bit by this a few times now, where on of our Dropshot servers is hardcoded to use the VERSION_INITIAL of its corresponding API. Changing the API itself works, but then the server doesn't actually support that version during testing. This changes the last spot I could find for this, in oximeter, to use the latest_version(). The NTP admin server is also hard-coded this way, but I'm fixing that separately in #10318.

@davepacheco
Copy link
Copy Markdown
Collaborator

Sorry for the trouble. Thanks for fixing it!

@bnaecker bnaecker merged commit b718b7a into main May 14, 2026
16 checks passed
@bnaecker bnaecker deleted the ben/use-latest-oximeter-version branch May 14, 2026 17:58
@sunshowers
Copy link
Copy Markdown
Contributor

Thanks for doing this!

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