Skip to content

test: update opentelemetry snapshots for svc_src tag#17552

Open
emmettbutler wants to merge 2 commits intomainfrom
emmett.butler/snapshot-svcsrc-otel
Open

test: update opentelemetry snapshots for svc_src tag#17552
emmettbutler wants to merge 2 commits intomainfrom
emmett.butler/snapshot-svcsrc-otel

Conversation

@emmettbutler
Copy link
Copy Markdown
Collaborator

Description

This change commits the result of running the opentelemetry test suite in my local testrunner container from #17502. That change removes the tag meta._dd.svc_src from the list of attributes ignored during snapshot testing. The change here is a small, easier-to-review subset of that change.

@emmettbutler emmettbutler requested a review from a team as a code owner April 15, 2026 15:46
@emmettbutler emmettbutler added the changelog/no-changelog A changelog entry is not required for this PR. label Apr 15, 2026
@cit-pr-commenter-54b7da
Copy link
Copy Markdown

Codeowners resolved as

tests/snapshots/tests.opentelemetry.test_context.test_otel_ddtrace_mixed_parenting.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_otel_multithreading.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_otel_span_parenting.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_otel_start_as_current_span_decorator_async.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_otel_trace_across_fork.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_otel_trace_multiple_coroutines.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_sampling_decisions_across_processes[manual.drop].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_context.test_sampling_decisions_across_processes[manual.keep].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_attributes.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_attributes_overrides[override0].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_attributes_overrides[override1].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_attributes_overrides[override2].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_attributes_overrides[override3].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_attributes_overrides[override4].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_events.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_span.test_otel_span_kind.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_distributed_trace_with_flask_app[with_ddtrace_run].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_distributed_trace_with_flask_app[with_opentelemetry_instrument].json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_otel_start_as_current_span_decorator.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_otel_start_current_span_with_default_args.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_otel_start_current_span_without_default_args.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_otel_start_span_ignore_exceptions.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_otel_start_span_record_exception.json  @DataDog/apm-python
tests/snapshots/tests.opentelemetry.test_trace.test_otel_start_span_without_default_args.json  @DataDog/apm-python

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 bot commented Apr 16, 2026

View all feedbacks in Devflow UI.

2026-04-16 13:56:35 UTC ℹ️ Start processing command /merge


2026-04-16 13:56:40 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 60m (p90).


2026-04-16 15:57:09 UTC 🚨 MergeQueue: This merge request is in error

error while getting head build completion result

Details

Error: There was an error while retrieving the result for pipeline 108010422

FullStacktrace:
child workflow execution error (type: mergequeue_private.MergeQueue_WaitForChecksOrUntilIsFinal, workflowID: 019d9694-631b-7312-add9-89e5f7f3d14c_63, runID: 019d9694-cd42-7150-b5d4-f3f8dd5e32b6, initiatedEventID: 63, startedEventID: 64): child workflow execution error (type: mergequeue.MergeQueue_WaitForCompletionOfRef, workflowID: 019d9694-cd42-7150-b5d4-f3f8dd5e32b6_8, runID: 019d9694-cdc5-79b9-9f31-842c3cd22929, initiatedEventID: 8, startedEventID: 10): There was an error while retrieving the result for pipeline 108010422 (type: FlowError, retryable: false): There was an error while retrieving the result for pipeline 108010422

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 bot commented Apr 16, 2026

View all feedbacks in Devflow UI.

2026-04-16 16:27:16 UTC ℹ️ Start processing command /merge


2026-04-16 16:27:22 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-04-16 17:06:12 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 60m (p90).


2026-04-16 19:06:40 UTCMergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build 0 took longer than expected. The current limit for the base branch 'main' is 120 minutes.

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 bot commented Apr 16, 2026

View all feedbacks in Devflow UI.

2026-04-16 21:31:46 UTC ℹ️ Start processing command /merge


2026-04-16 21:31:51 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 60m (p90).


2026-04-16 22:06:04 UTC ⚠️ MergeQueue: This merge request build was cancelled

emmett.butler@datadoghq.com cancelled this merge request build

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge -c

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 bot commented Apr 16, 2026

View all feedbacks in Devflow UI.

2026-04-16 22:05:59 UTC ℹ️ Start processing command /merge -c

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 bot commented Apr 16, 2026

View all feedbacks in Devflow UI.

2026-04-16 22:06:48 UTC ℹ️ Start processing command /merge
Use /merge -c to cancel this operation!


2026-04-16 22:06:52 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 60m (p90).

Use /merge -c to cancel this operation!


⏳ Processing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog A changelog entry is not required for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants