system-tests icon indicating copy to clipboard operation
system-tests copied to clipboard

Change reporting from otel_collector to generic for the Otel Scenario

Open wantsui opened this issue 3 months ago • 1 comments

Motivation

The reports are uploaded to the backend but @bojbrook found that the unexpected otel_collector type fails the update.

Changes

This goal of this change is to have a generic value for this field instead so @colin-higgins can make a more generic backend change.

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • [ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
  • [ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
  • [ ] A docker base image is modified?
    • [ ] the relevant build-XXX-image label is present
  • [ ] A scenario is added (or removed)?

wantsui avatar Nov 14 '25 19:11 wantsui

CODEOWNERS have been resolved as:

tests/otel_postgres_metrics_e2e/test_postgres_metrics.py                @DataDog/system-tests-core
utils/_features.py                                                      @DataDog/system-tests-core

github-actions[bot] avatar Nov 14 '25 19:11 github-actions[bot]

/merge

quinna-h avatar Nov 18 '25 20:11 quinna-h

View all feedbacks in Devflow UI.

2025-11-18 20:02:14 UTC :information_source: Start processing command /merge


2025-11-18 20:02:26 UTC :information_source: 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. 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.


2025-11-18 20:09:33 UTC :information_source: MergeQueue: This merge request was already merged

This pull request was merged directly.