Skip to content

Add Cancel Query section to DBM docs #29688

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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

newkek
Copy link

@newkek newkek commented Jun 2, 2025

What does this PR do? What is the motivation?

Documentation for the new Cancel Query functionality in Database Monitoring

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

To have your PR automatically merged after it receives the required reviews, add the following PR comment:

/merge

Additional notes

@newkek newkek self-assigned this Jun 2, 2025
Copy link
Contributor

github-actions bot commented Jun 2, 2025

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

@hestonhoffman hestonhoffman self-assigned this Jun 2, 2025
@newkek newkek marked this pull request as ready for review June 3, 2025 14:06
@newkek newkek requested review from a team as code owners June 3, 2025 14:06
@github-actions github-actions bot added the Images Images are added/removed with this PR label Jun 3, 2025
@newkek newkek requested a review from hestonhoffman June 3, 2025 14:13
Copy link
Contributor

@hestonhoffman hestonhoffman left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I left you some edits

@newkek newkek requested a review from hestonhoffman June 3, 2025 16:04
@hestonhoffman
Copy link
Contributor

If this is good to go, feel free to /merge. We've been having some issues with the merge queue though, so I'll keep an eye on it and just manually merge if necessary.

@hestonhoffman hestonhoffman added the Do Not Merge Just do not merge this PR :) label Jun 3, 2025
Copy link
Contributor

@hestonhoffman hestonhoffman left a comment

Choose a reason for hiding this comment

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

One more suggestion. Realized I missed something

@newkek newkek requested a review from hestonhoffman June 3, 2025 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Just do not merge this PR :) Images Images are added/removed with this PR okr11
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants