Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reflectiz integration fix facet reflectiz.alert.severity source (ECOINT-23) #2623

Merged
merged 45 commits into from
Mar 12, 2025

Conversation

elie-b-ref
Copy link
Contributor

What does this PR do?

The Reflectiz Integration supports a facet called reflectiz.alert.severity today the course of it is "log", it should be "tag" since this information is only provided as a tag

Motivation

The current facet was not working when we were trying to filter by it because of the wrong source

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

elie-b-ref and others added 30 commits January 16, 2025 09:45
Co-authored-by: Sandra (neko) <165049174+neko-dd@users.noreply.github.com>
Co-authored-by: Sandra (neko) <165049174+neko-dd@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Sandra (neko) <165049174+neko-dd@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
@dd-dominic dd-dominic requested review from a team and JoshPatel13 and removed request for a team March 10, 2025 15:25
@dd-dominic dd-dominic changed the title Reflectiz integration fix facet reflectiz.alert.severity source Reflectiz integration fix facet reflectiz.alert.severity source (ECOINT-23) Mar 10, 2025
@elie-b-ref
Copy link
Contributor Author

I updated the PR with provided comments, thanks

@audesikorav audesikorav added the assets/deploy-logs-staging ONLY USED BY Logs Backend - Validates that a PR is OK to go to staging label Mar 11, 2025
@JoshPatel13 JoshPatel13 added this pull request to the merge queue Mar 12, 2025
Merged via the queue into DataDog:master with commit 72b26f4 Mar 12, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets/deploy-logs-staging ONLY USED BY Logs Backend - Validates that a PR is OK to go to staging ecosystems/review-requested kind/technology-partner product/approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants