Skip to content

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Feb 6, 2025

Transport version changes must be unique per branch. Some transport version changes meant for 9.0 are missing unique backport constants. This is a backport of #121985, adding unique transport version patch numbers for each change intended for 9.0.

Transport version changes must be unique per branch. Some transport
version changes meant for 9.0 are missing unique backport constants.
This is a backport of elastic#121985, adding unique transport version patch
numbers for each change intended for 9.0.
@rjernst rjernst added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 7, 2025
@elasticsearchmachine elasticsearchmachine merged commit f5d3f38 into elastic:9.0 Feb 7, 2025
16 checks passed
@rjernst rjernst deleted the fix_transport_versions_9.0 branch February 7, 2025 01:10
public static final TransportVersion TRANSPORT_STATS_HANDLING_TIME_REQUIRED_90 = def(9_000_0_02);
public static final TransportVersion REMOVE_DESIRED_NODE_VERSION_90 = def(9_000_0_03);
public static final TransportVersion ESQL_DRIVER_TASK_DESCRIPTION_90 = def(9_000_0_04);

Copy link
Contributor

Choose a reason for hiding this comment

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

For a follow up: insert a comment here that versions >= 9_001_0_00 are forbidden here? So that any auto-backport breaks, and manual backports should read it (as it would be right on the conflict line?)

Copy link
Member

Choose a reason for hiding this comment

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

There could always be a test that failed if the transport version was ever 9_001_0_00 or above

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, that was what @alexey-ivanov-es proposed too. I'm going to add that.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Transport API Transport client API >non-issue v9.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants