Skip to content

Fix duplicate Maven publication coordinates#19294

Merged
trask merged 2 commits into
open-telemetry:mainfrom
trask:fix/duplicate-maven-publication-coordinates
Jul 22, 2026
Merged

Fix duplicate Maven publication coordinates#19294
trask merged 2 commits into
open-telemetry:mainfrom
trask:fix/duplicate-maven-publication-coordinates

Conversation

@trask

@trask trask commented Jul 22, 2026

Copy link
Copy Markdown
Member

The 2.30.0 Maven Central deployment failed because the Sofa RPC library and library-autoconfigure projects published different artifacts using the same Maven coordinates. Parallel publication into the shared release repository produced mismatched JARs, checksums, and signatures.

This change:

  • gives the Sofa RPC autoconfigure artifact the expected -autoconfigure suffix
  • adds a configuration-cache-compatible task that rejects duplicate Maven publication coordinates
  • runs the guard during ordinary check tasks and before every PublishToMavenRepository task

@trask
trask marked this pull request as ready for review July 22, 2026 16:17
@trask
trask requested a review from a team as a code owner July 22, 2026 16:18
Copilot AI review requested due to automatic review settings July 22, 2026 16:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes conflicting Maven coordinates that caused the 2.30.0 deployment failure.

Changes:

  • Adds the expected SOFARPC autoconfigure artifact suffix.
  • Detects duplicate Maven publication coordinates.
  • Runs validation during checks and repository publication.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
instrumentation/sofa-rpc-5.4/library-autoconfigure/build.gradle.kts Assigns unique autoconfigure coordinates.
conventions/src/main/kotlin/io/opentelemetry/instrumentation/gradle/CheckMavenPublicationCoordinatesTask.kt Implements duplicate-coordinate validation.
build.gradle.kts Collects coordinates and wires validation into checks and publishing.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 22, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-22 18:28:03 UTC.

  • Status: Merged.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

@TaskAction
fun checkPublicationCoordinates() {
val duplicatePublications = publicationCoordinates.get()
.groupBy { it.substringBefore('=') }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

stringBefore!!! I love that.

@trask
trask merged commit f5727bd into open-telemetry:main Jul 22, 2026
183 of 187 checks passed
@trask
trask deleted the fix/duplicate-maven-publication-coordinates branch July 22, 2026 18:24
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.

4 participants