Skip to content

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Sep 8, 2025

This commit adds a new generateInitialTransportVersion task to gradle. This task will be called by release automation to ensure each new release version of Elasticsearch has a unique transport version id.

This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
@rjernst rjernst requested a review from a team as a code owner September 8, 2025 21:10
@rjernst rjernst added >non-issue :Delivery/Build Build or test infrastructure auto-backport Automatically create backport pull requests when merged v9.1.4 v9.0.7 v8.18.7 v8.19.4 labels Sep 8, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Delivery Meta label for Delivery team label Sep 8, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@rjernst rjernst requested review from a team as code owners September 9, 2025 18:27
@rjernst rjernst force-pushed the transport/generate-initial branch from 0085d86 to 4118497 Compare September 9, 2025 18:28
project.getPluginManager().apply(VersionPropertiesPlugin.class);
var psService = project.getPlugins().apply(ProjectSubscribeServicePlugin.class).getService();

Properties versions = (Properties) project.getExtensions().getByName(VersionPropertiesPlugin.VERSIONS_EXT);
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought we were going to use VersionPropertiesBuildService here?

Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

Minor comments. Otherwise LGTM.

t.setGroup(taskGroup);
t.setDescription("(Re)generates an initial transport version for an Elasticsearch release version");

System.out.println("Setting current version: " + currentVersion);
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this just for local debugging? Can remove this or replace it with a proper logger usage?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, removed

@rjernst rjernst merged commit f105436 into elastic:main Sep 9, 2025
35 checks passed
@rjernst rjernst deleted the transport/generate-initial branch September 9, 2025 23:18
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Sep 9, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Sep 9, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.1 Commit could not be cherrypicked due to conflicts
9.0
8.18
8.19

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 134333

rjernst added a commit to rjernst/elasticsearch that referenced this pull request Sep 9, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
elasticsearchmachine pushed a commit that referenced this pull request Sep 10, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
elasticsearchmachine pushed a commit that referenced this pull request Sep 11, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Sep 12, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
elasticsearchmachine pushed a commit that referenced this pull request Sep 12, 2025
* Add initial transport version generation task (#134333)

This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <[email protected]>
elasticsearchmachine pushed a commit that referenced this pull request Sep 12, 2025
This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
sarog pushed a commit to portsbuild/elasticsearch that referenced this pull request Sep 19, 2025
…ic#134410)

This commit adds a new generateInitialTransportVersion task to gradle.
This task will be called by release automation to ensure each new
release version of Elasticsearch has a unique transport version id.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v8.18.7 v8.19.4 v9.0.7 v9.1.4 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants