Skip to content

Fix versioning discrepancy between sled-agent API and types crate#9901

Merged
jgallagher merged 1 commit intomainfrom
john/sled-agent-types-versions-offby1
Feb 23, 2026
Merged

Fix versioning discrepancy between sled-agent API and types crate#9901
jgallagher merged 1 commit intomainfrom
john/sled-agent-types-versions-offby1

Conversation

@jgallagher
Copy link
Contributor

As it evolved, #9570 made various changes to both the sled-agent types crate and the sled-agent API. As it landed, though, it made one versioning bump to sled-agent API (v20 - BGP) and two to the types crate (v20 - lockstep API, v21 - BGP). This squishes the latter two down to just one (v20 - BGP) for consistency with the API's versioning.

As it evolved, #9570 made various changes to both the sled-agent types
crate and the sled-agent API. As it landed, though, it made one
versioning bump to sled-agent API (v20 - BGP) and two to the types crate
(v20 - lockstep API, v21 - BGP). This squishes the latter two down to
just one (v20 - BGP) for consistency with the API's versioning.
@karencfv
Copy link
Contributor

Hm, interesting. I thought #9896 would have caused some sort of merge conflict somehow, but it didn't. Nice!

Copy link
Contributor

@sunshowers sunshowers left a comment

Choose a reason for hiding this comment

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

good catch, thanks!

@jgallagher
Copy link
Contributor Author

Hm, interesting. I thought #9896 would have caused some sort of merge conflict somehow, but it didn't. Nice!

I think because that only changed API versions, not the types crate, and this PR only changed the types crate, not API versions. 😅

@jgallagher jgallagher merged commit af8c9b3 into main Feb 23, 2026
16 checks passed
@jgallagher jgallagher deleted the john/sled-agent-types-versions-offby1 branch February 23, 2026 23:14
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