Skip to content

opentelemetry-instrumentation-cassandra: implement semconv stability opt-in#4747

Open
SurbhiD404 wants to merge 6 commits into
open-telemetry:mainfrom
SurbhiD404:semconv-migration-cassandra
Open

opentelemetry-instrumentation-cassandra: implement semconv stability opt-in#4747
SurbhiD404 wants to merge 6 commits into
open-telemetry:mainfrom
SurbhiD404:semconv-migration-cassandra

Conversation

@SurbhiD404

@SurbhiD404 SurbhiD404 commented Jun 26, 2026

Copy link
Copy Markdown

Description

Implements semantic convention stability support for the Cassandra instrumentation by adding support for OTEL_SEMCONV_STABILITY_OPT_IN=database.

Changes

  • Add support for semantic convention stability opt-in for database signals.

  • Conditionally emit legacy or stable database semantic convention attributes using _report_old() and _report_new().

  • Preserve existing behavior when no opt-in mode is configured.

  • Replace the hardcoded schema URL with _get_schema_url_for_signal_types().

  • Add unit tests covering:

    • default (legacy) behavior,
    • OTEL_SEMCONV_STABILITY_OPT_IN=database,
    • OTEL_SEMCONV_STABILITY_OPT_IN=database/dup.

Part of #2453

Fixes #4714

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Ran the Cassandra instrumentation test suite:
pytest instrumentation/opentelemetry-instrumentation-cassandra/tests -v

Result:

11 passed

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@xrmx xrmx 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.

Please add a changelog entry as well

@github-project-automation github-project-automation Bot moved this to Reviewed PRs that need fixes in Python PR digest Jun 26, 2026
@SurbhiD404 SurbhiD404 force-pushed the semconv-migration-cassandra branch from 428cfd0 to aafb333 Compare June 27, 2026 13:31
@SurbhiD404 SurbhiD404 force-pushed the semconv-migration-cassandra branch from f59f4eb to 6aee203 Compare June 30, 2026 04:54
@xrmx xrmx moved this from Reviewed PRs that need fixes to Ready for review in Python PR digest Jun 30, 2026
@xrmx xrmx moved this from Ready for review to Reviewed PRs that need fixes in Python PR digest Jun 30, 2026
@xrmx

xrmx commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

@SurbhiD404 Have you read CONTRIBUTING.md? You are supposed to setup pre-commit locally to avoid a bunch of failures in CI.

@SurbhiD404 SurbhiD404 force-pushed the semconv-migration-cassandra branch from 9d19d84 to fd4370c Compare July 1, 2026 05:14
@github-project-automation github-project-automation Bot moved this from Reviewed PRs that need fixes to Approved PRs in Python PR digest Jul 1, 2026
@xrmx xrmx changed the title feat(cassandra): implement semconv stability opt-in opentelemetry-instrumentation-cassandra: implement semconv stability opt-in Jul 2, 2026
Signed-off-by: Surbhi Dharvan <dharvansurbhi@gmail.com>
Signed-off-by: Surbhi Dharvan <dharvansurbhi@gmail.com>
Signed-off-by: Surbhi Dharvan <dharvansurbhi@gmail.com>
Signed-off-by: Surbhi Dharvan <dharvansurbhi@gmail.com>
…view

Signed-off-by: Surbhi Dharvan <dharvansurbhi@gmail.com>
Signed-off-by: Surbhi Dharvan <dharvansurbhi@gmail.com>
@SurbhiD404 SurbhiD404 force-pushed the semconv-migration-cassandra branch from 2e7f7cc to 0f5e1b9 Compare July 3, 2026 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved PRs

Development

Successfully merging this pull request may close these issues.

opentelemetry-instrumentation-cassandra: semantic convention stability migration

3 participants