This pull request has been designated for reprocessing. Please check back in a few minutes.

Make transport versions CSV update idempotent #159744

Open
mark-vieira opened 4:21pm on September 18, 2026 wants to merge 86 Ξ” into elastic/elasticsearch main from
transport-versions-csv-idempotency

Pull Request Overview

  • Opened on September 18, 2026
  • Status Open
  • Commit count 3 with first commit September 18, 2026

Total Delta

86 Total Diff Delta

Open Days

Open 0 weekdays

Test Delta

0 Diff Delta in Test Files
Breakdown by Phase

How long has this pull request spent in each phase of its lifecycle?

Fraction of total time Business days Phase
 
0.0 days Authoring 1 commit before pull request opened for review
 
0.0 days Awaiting first review
 
0.7 days Revising work with 2 commits in response to 2 reviews that left 6 comments

Total time for pull request still awaiting merge: 0.7 business days

Author avatar

Make transport versions CSV update idempotent

The idempotency check in updateTransportVersionsCSV was flawed in that it was comparing the transport version against the incremented one which will never end up matching. Instead check that it doesn't exceed the current upper bound. This allows partially completed release steps to continue if we've already recorded the released transport version into transportversions.csv but later steps failed and we are performing a retry.

Comments Threads Pending Resolution

Resolved Comment Threads

rjernst reviewed on September 18, 2026
rjernst reviewed on September 18, 2026
build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/transport/UpdateTransportVersionsCSVTask.java
Outdated
51
        // Idempotency check to ensure we don't try to increment this again if there is already an entry in the CSV file for
52
        // the given stack version.
53
        Integer existingTransportVersionId = getExistingTransportVersionId(stackVersion);
54
        if (existingTransportVersionId != null) {
55
            if (existingTransportVersionId > upperBoundId) {