Skip to content

Fix tag syntax in monitor filtering instructions - #40454

Open
mohamedwidar wants to merge 2 commits into
DataDog:masterfrom
mohamedwidar:patch-1
Open

mohamedwidar wants to merge 2 commits into
DataDog:masterfrom
mohamedwidar:patch-1

Conversation

@mohamedwidar

@mohamedwidar mohamedwidar commented Oct 6, 2026 •

Copy link
Copy Markdown

added double quotes to the example since it conforms with the photo example and my testing in datadog environments

What does this PR do? What is the motivation?

Merge readiness

this is a simple docs edit ready to be merged immediately

  • Ready for merge

AI assistance

No AI was used for this suggestion

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

Additional notes

added double quotes to the example since it conforms with the photo example and my testing in datadog environments
@mohamedwidar
mohamedwidar requested a review from a team as a code owner October 6, 2026 07:49
@jeff-morgan-dd jeff-morgan-dd self-assigned this Oct 6, 2026

@jeff-morgan-dd jeff-morgan-dd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR! Can you please double-check the syntax for the exclude portion - your update differs from the screenshot and will search for that exact string inside of the quotes.

@mohamedwidar

Copy link
Copy Markdown
Author

Thanks for the PR! Can you please double-check the syntax for the exclude portion - your update differs from the screenshot and will search for that exact string inside of the quotes.

That is really interesting coz for me it only works when there is brackets around it :)

image image

my original version is indeed wrong

negation needs round brackets
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