Skip to content

DOCS-14165: SDS Logs Findings explorer#37797

Open
drichards-87 wants to merge 12 commits into
masterfrom
deforest/DOCS-14165-sds-log-findings-explorer
Open

DOCS-14165: SDS Logs Findings explorer#37797
drichards-87 wants to merge 12 commits into
masterfrom
deforest/DOCS-14165-sds-log-findings-explorer

Conversation

@drichards-87

@drichards-87 drichards-87 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

DOCS-14165.

This is a small SDS docs update for the new Logs Findings explorer.

Review focus:

  • Adds one new Logs tab to the Investigate Sensitive Data Findings guide with steps for using the updated Logs Findings explorer.
  • Keeps the existing non-log telemetry guidance on a renamed APM, RUM, and Events tab.
  • Updates a screenshot on the SDS overview page.
  • Adds a new screenshot on the Investigate Sensitive Data Findings page.

Merge readiness

  • Ready for merge

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.

AI assistance

Used Codex to draft the PR description.

Additional notes

@drichards-87 drichards-87 added WORK IN PROGRESS No review needed, it's a wip ;) Do Not Merge Just do not merge this PR :) labels Jun 26, 2026
@datadog-prod-us1-3

This comment has been minimized.

@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added the Images Images are added/removed with this PR label Jun 26, 2026
@drichards-87 drichards-87 marked this pull request as ready for review June 29, 2026 22:53
@drichards-87 drichards-87 requested a review from a team as a code owner June 29, 2026 22:53
@drichards-87 drichards-87 removed WORK IN PROGRESS No review needed, it's a wip ;) Do Not Merge Just do not merge this PR :) labels Jul 1, 2026

@domalessi domalessi 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.

A few comments in line. Nothing blocking!

{{% tab "Telemetry Data" %}}
{{% tab "Logs" %}}

The Logs Findings explorer is an updated experience for investigating log findings. To access it, click {{< ui >}}Try the new version{{< /ui >}} in the blue banner at the top of the Telemetry tab. APM, RUM, and Events findings are not available in this explorer. To view those findings, click {{< ui >}}Go back{{< /ui >}} in the banner.

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.

Just noting that you reference the UI tab as Telemetry here but Sensitive Data Rule Findings for "APM, RUM, and Events" and "Cloud Storage" tabs on the docs page. Is that intentional, or worth updating for consistency?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch. It was intentional. We're starting with a new Logs explorer, but later we'll have separate explorers for APM and RUM, as well. Technically, logs are a type of telemetry, so the PM and I agreed to name the second tab "APM, RUM, and Events" instead of "Telemetry".

Comment thread content/en/security/sensitive_data_scanner/_index.md Outdated
Co-authored-by: domalessi <111786334+domalessi@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images Images are added/removed with this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants