Skip to content

Fixed v1 DBM boolean tag assertion.#7096

Merged
AlexeyKuznetsov-DD merged 1 commit into
mainfrom
alexeyk/v1-fix-dbm
Jun 5, 2026
Merged

Fixed v1 DBM boolean tag assertion.#7096
AlexeyKuznetsov-DD merged 1 commit into
mainfrom
alexeyk/v1-fix-dbm

Conversation

@AlexeyKuznetsov-DD
Copy link
Copy Markdown
Contributor

Motivation

Fixed v1 DBM boolean tag assertion.

Changes

Used is_same_boolean for safe comparison with true as in v1 dd-trace-java can now return real Boolean.

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

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

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • Anything but tests/ or manifests/ is modified ? 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, removed or renamed?

@AlexeyKuznetsov-DD AlexeyKuznetsov-DD self-assigned this Jun 5, 2026
@AlexeyKuznetsov-DD AlexeyKuznetsov-DD requested a review from a team as a code owner June 5, 2026 14:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2026

CODEOWNERS have been resolved as:

tests/integrations/test_dbm.py                                          @DataDog/system-tests-core

@AlexeyKuznetsov-DD AlexeyKuznetsov-DD merged commit 06047fc into main Jun 5, 2026
428 checks passed
@AlexeyKuznetsov-DD AlexeyKuznetsov-DD deleted the alexeyk/v1-fix-dbm branch June 5, 2026 15:01
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