Skip to content

chore(ai-guard): enable event tag in node#6779

Open
manuel-alvarez-alvarez wants to merge 3 commits intomainfrom
malvarez/ai-guard-event-tag-manifest
Open

chore(ai-guard): enable event tag in node#6779
manuel-alvarez-alvarez wants to merge 3 commits intomainfrom
malvarez/ai-guard-event-tag-manifest

Conversation

@manuel-alvarez-alvarez
Copy link
Copy Markdown
Member

@manuel-alvarez-alvarez manuel-alvarez-alvarez commented Apr 21, 2026

Motivation

Support for the event tag on the root span has been added in Node.js.

Changes

Enable the corresponding test in Node.js.

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?

@manuel-alvarez-alvarez manuel-alvarez-alvarez force-pushed the malvarez/ai-guard-event-tag-manifest branch from 3c38352 to 2b8e2e1 Compare April 21, 2026 12:59
@github-actions
Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

manifests/nodejs.yml                                                    @DataDog/dd-trace-js

@manuel-alvarez-alvarez manuel-alvarez-alvarez marked this pull request as ready for review April 21, 2026 12:59
@datadog-prod-us1-3
Copy link
Copy Markdown
Contributor

datadog-prod-us1-3 Bot commented Apr 21, 2026

Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 2f1b1f3 | Docs | Datadog PR Page | Give us feedback!

Copy link
Copy Markdown
Member

@BridgeAR BridgeAR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with the change (we released a new version in-between)

Comment thread manifests/nodejs.yml Outdated
manuel-alvarez-alvarez and others added 2 commits April 27, 2026 10:33
Co-authored-by: Ruben Bridgewater <ruben.bridgewater@datadoghq.com>
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.

4 participants