Skip to content

Activate messaging v1.43 in common v3 preview#19276

Draft
trask wants to merge 17 commits into
open-telemetry:mainfrom
trask:messaging-v1-43-v3-activation
Draft

Activate messaging v1.43 in common v3 preview#19276
trask wants to merge 17 commits into
open-telemetry:mainfrom
trask:messaging-v1-43-v3-activation

Conversation

@trask

@trask trask commented Jul 19, 2026

Copy link
Copy Markdown
Member

Depends on #19269, #19271, #19272, #19273, #19274, and #19275 (#19271 includes #19270).

Temporary integration baseline: 909cbe17c7f633b6464e5330e094547b216d2d23.
Activation-only review range: 909cbe17c7f633b6464e5330e094547b216d2d23..1c34d56c63e7d2edacf3c4045863db8161bcf3c6.

This PR has exactly one activation-owned commit above a fork-only integration branch constructed from current upstream main (03670ca6bf83d9551d53b948e498e1ae01f1c1c5) and the exact final family heads:

  • AWS: 677a8eb7467c1edaaf79cfa72dae9805e54ad695
  • Kafka ecosystem (including core): cac11bfe6b0bc12e4c0df68046406280eb5f61ba
  • JMS/NATS/Spring: 1a2d8820ffabbeaf6d8fa02fbe806cd9ba5b9004
  • Pulsar: 2fe52dc7cb1a23690d1af7b8270c50914971daed
  • RabbitMQ: 8a9f68d1154772e955713bd4b68e20415896e3dc
  • RocketMQ: a9282cc1f0a1de80f646fd21b3b5ecaa4915b642

messaging-v1-43-integration must never be merged upstream. After dependency PRs merge, this branch will be rebuilt from current main with only its activation delta reapplied.

Scope

  • Activates messaging v1.43 under common v3 preview while preserving explicit preview and /dup behavior.
  • Deprecates MessageOperation after integrated production callers migrate.
  • Uses distinct typed span-name factory names, preserving legacy null-literal source compatibility.
  • Adds the Unreleased deprecation changelog entry and narrowly scoped compatibility suppressions.

Family telemetry behavior remains owned by dependency PRs; activation-range production updates are mechanical typed factory-name changes.

Validation

  • ./gradlew :instrumentation:rabbitmq-2.7:javaagent:check spotlessCheck :instrumentation-api-incubator:test compileJava --console=plain
  • AWS SDK 2.2, Spring Integration, Rabbit/Spring Rabbit, and RocketMQ owner checks pass.
  • Current upstream main and every exact family leaf are ancestors of the integration baseline.
  • Activation range contains exactly one commit, passes git diff --check, and the worktree is clean.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 19, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-22 20:51:38 UTC.

  • Waiting on: Author
  • Next step: Move out of draft to request review.

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

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

Activates messaging v1.43 telemetry by default in common v3 preview and deprecates the legacy operation API.

Changes:

  • Selects v1 experimental messaging semantic conventions under v3 preview.
  • Deprecates MessageOperation and compatibility factories.
  • Updates selection tests and changelog documentation.

Reviewed changes

Copilot reviewed 178 out of 178 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
CHANGELOG.md Records the deprecation.
MessageOperation.java Deprecates the legacy enum.
MessagingAttributesExtractor.java Deprecates legacy factories.
MessagingSpanKindExtractor.java Deprecates the legacy overload.
MessagingSpanNameExtractor.java Deprecates legacy factories.
MessagingAttributesExtractorTest.java Suppresses intentional deprecated usage.
SemconvSelectionResolver.java Activates messaging v1.43 for v3 preview.
SemconvStabilityTest.java Updates the selection matrix.

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

Copilot reviewed 178 out of 178 changed files in this pull request and generated 1 comment.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from 88b4d0d to 9cded86 Compare July 22, 2026 16:11
@trask
trask requested a review from Copilot July 22, 2026 16:11

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

Copilot reviewed 198 out of 198 changed files in this pull request and generated no new comments.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from 9cded86 to be32bf4 Compare July 22, 2026 16:16
@trask
trask requested a review from Copilot July 22, 2026 16:17

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

Copilot reviewed 198 out of 198 changed files in this pull request and generated 1 comment.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from be32bf4 to c20c14a Compare July 22, 2026 17:32
@trask
trask requested a review from Copilot July 22, 2026 17:33

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

Copilot reviewed 198 out of 198 changed files in this pull request and generated 1 comment.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from c20c14a to 686014a Compare July 22, 2026 17:52
@trask
trask force-pushed the messaging-v1-43-v3-activation branch from 686014a to 420b420 Compare July 22, 2026 18:03
@trask
trask requested a review from Copilot July 22, 2026 18:04

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

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

Comment thread instrumentation/spring/spring-rabbit-1.0/javaagent/build.gradle.kts Outdated
Comment thread instrumentation/rabbitmq-2.7/javaagent/build.gradle.kts Outdated
@trask
trask force-pushed the messaging-v1-43-v3-activation branch from 420b420 to 28b2163 Compare July 22, 2026 18:33
@trask
trask requested a review from Copilot July 22, 2026 18:33

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

Copilot reviewed 199 out of 199 changed files in this pull request and generated 1 comment.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from 28b2163 to 7fa825e Compare July 22, 2026 19:34
@trask
trask requested a review from Copilot July 22, 2026 19:34

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

Copilot reviewed 199 out of 199 changed files in this pull request and generated no new comments.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from 7fa825e to e427401 Compare July 22, 2026 20:26
@trask
trask requested a review from Copilot July 22, 2026 20:26

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

Copilot reviewed 200 out of 200 changed files in this pull request and generated no new comments.

@trask
trask force-pushed the messaging-v1-43-v3-activation branch from e427401 to 1c34d56 Compare July 22, 2026 20:44
@trask
trask requested a review from Copilot July 22, 2026 20:44

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

Copilot reviewed 200 out of 200 changed files in this pull request and generated no new comments.

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.

2 participants