-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Revise security findings migration documentation #33623
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
base: master
Are you sure you want to change the base?
Conversation
Updated the security findings migration guide to clarify the changes in query syntax and required actions for users. Added details about the new querying syntax and the timeline for deprecating legacy API endpoints.
Preview links (active after the
|
janine-c
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
![]()
|
|
||
| This change comes with a set of [new features](#new-features) that use the new schema, and may also impact your existing workflows. This page details what is changing, the workflows that Datadog automatically updates, and the changes you need to make to avoid interruptions in your existing workflows. | ||
|
|
||
| <div class="alert alert-warning">Datadog will start rolling out changes January 28, 2026. You should plan to update any affected workflows in the first half of 2026 to avoid any interruptions as Datadog deprecates the old syntax. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: "workflows" in this context is meant in a broad general way, but it can be confused with workflows from Datadog Workflow Automation IMO. I don't have a great replacement though, maybe something like "configurations"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I can understand your concern. @etienneddog what do you think? I went back to your doc and saw that we reference both capital-W and lowercase-w workflows, and both are common in the docs as well. Do you think we need to worry about the lowercase-w ones causing confusion here?
|
I agree - feel free to iterate on it (on my phone for the next three days)
…On Wed 31 Dec 2025 at 10:02, Hugo Queinnec ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In content/en/security/guide/security-findings-migration.md
<#33623 (comment)>
:
> @@ -22,28 +22,44 @@ To make it easier to search for security findings throughout Datadog, the syntax
This change comes with a set of [new features](#new-features) that use the new schema, and may also impact your existing workflows. This page details what is changing, the workflows that Datadog automatically updates, and the changes you need to make to avoid interruptions in your existing workflows.
+<div class="alert alert-warning">Datadog will start rolling out changes January 28, 2026. You should plan to update any affected workflows in the first half of 2026 to avoid any interruptions as Datadog deprecates the old syntax.
nit: "workflows" in this context is meant in a broad general way, but it
can be confused with workflows from Datadog Workflow Automation IMO. I
don't have a great replacement though, maybe something like
"configurations"?
—
Reply to this email directly, view it on GitHub
<#33623 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3RDZOUGAM7X23TYZL2T4RD4EOGIZAVCNFSM6AAAAACQKRGMJSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTMMRQGUYDKOBYGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Updated the security findings migration guide to clarify the changes in query syntax and required actions for users. Added details about the new querying syntax and the timeline for deprecating legacy API endpoints.
What does this PR do? What is the motivation?
Merge instructions
Merge readiness:
For Datadog employees:
Your branch name MUST follow the
<name>/<description>convention and include the forward slash (/). Without this format, your pull request will not pass 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.
[6/5/2025] Merge queue has been disabled on the documentation repo. If you have write access to the repo, the PR has been reviewed by a Documentation team member, and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #documentation channel in Slack.
Additional notes