Skip to content

NOOP some transport versions #127037

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

Merged
merged 2 commits into from
Apr 17, 2025
Merged

NOOP some transport versions #127037

merged 2 commits into from
Apr 17, 2025

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Apr 17, 2025

We accidentally released the 9_051_0_00 and 9_052_0_00 transport versions to serverless without their backing code. We can't take that back. It's live. You can't unbake the cake.

That version of Elasticsearch will claim to be 9_052_0_00 but actually speak 9_050_0_00 code. To make main compatible with that version we're bumping all transport versions in main past those numbers.

This is a forbidden action. You may not do this. I may not do this. It is not safe. But it is the only way to make the world whole again.

In all seriousness, everything will be fine. It's against the rules but sometimes you have to break rules to put things right. We believe some serverless dev environments may break which is sad but it's worth it to bring everything back to sane land.

We accidentally released the `9_051_0_00` and `9_052_0_00` transport
versions to serverless without their backing code. We can't take that
back. It's live. You can't unbake the cake.

That version of Elasticsearch will claim to be `9_052_0_00` but actually
speak `9_050_0_00` code. To make main compatible with that version we're
bumping all transport versions in main *past* those numbers.

This is a forbidden action. You may not do this. I may not do this. It
is not safe. But it is the only way to make the world whole again.

In all seriousness, everything will be fine. It's against the rules but
sometimes you have to break rules to put things right. We believe some
serverless dev environments may break which is sad but it's worth it to
bring everything back to sane land.
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Apr 17, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM.

It's against the rules

TBC it's against the rules for released versions, it just so happens that this was already released to serverless.

@nik9000 nik9000 enabled auto-merge (squash) April 17, 2025 22:38
@nik9000 nik9000 merged commit 492b036 into elastic:main Apr 17, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Transport API Transport client API >non-issue Team:Core/Infra Meta label for core/infra team v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants