Skip to content

Deprecate PCI DSS for Logs and APM #30919

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Aug 8, 2025
Merged

Deprecate PCI DSS for Logs and APM #30919

merged 10 commits into from
Aug 8, 2025

Conversation

estherk15
Copy link
Contributor

@estherk15 estherk15 commented Aug 5, 2025 •
edited
Loading

What does this PR do? What is the motivation?

Remove docs for PCI DSS for Logs and APM. There is an effort to standardize the Datadog platform and we are no longer supporting product specific compliance.
DOCS-11720

  • Update the Data Security PCI Compliance page
  • Remove PCI configuration instructions in APM and Logs
  • Remove left nav menu items for PCI DSS under Logs
  • Remove reference to intake agent-http-intake-pci.logs.datadoghq.com and usePciIntake
  • Delete APM and Logs PCI configuration shortcodes

Merge instructions

Merge readiness:

  • Ready for merge

@estherk15 estherk15 added the WORK IN PROGRESS No review needed, it's a wip ;) label Aug 5, 2025
@estherk15 estherk15 requested review from a team as code owners August 5, 2025 20:17
@github-actions github-actions bot added Architecture Everything related to the Doc backend Guide Content impacting a guide labels Aug 5, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 5, 2025 •
edited
Loading

✅ Documentation Team Review

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 5, 2025 •
edited
Loading

Preview links (active after the build_preview check completes)

Modified Files

@estherk15 estherk15 added the Do Not Merge Just do not merge this PR :) label Aug 6, 2025
estherk15
estherk15 commented Aug 7, 2025
View reviewed changes
@estherk15 estherk15 requested a review from jinatdatadog August 7, 2025 22:33
@estherk15 estherk15 removed the WORK IN PROGRESS No review needed, it's a wip ;) label Aug 7, 2025
jinatdatadog
jinatdatadog approved these changes Aug 8, 2025
View reviewed changes
Copy link
Contributor

@jinatdatadog jinatdatadog left a comment

Choose a reason for hiding this comment

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

one reply to comment, otherwise LGTM!

estherk15
estherk15 commented Aug 8, 2025
View reviewed changes
@estherk15 estherk15 removed the Do Not Merge Just do not merge this PR :) label Aug 8, 2025
bcaudan
bcaudan approved these changes Aug 8, 2025
View reviewed changes
Copy link
Contributor

@bcaudan bcaudan left a comment

Choose a reason for hiding this comment

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

LGTM for browser logs

drichards-87
drichards-87 approved these changes Aug 8, 2025
View reviewed changes
Copy link
Contributor

@drichards-87 drichards-87 left a comment

Choose a reason for hiding this comment

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

Leaving some minor feedback on the PR. Approving now so you can merge afterwards.

@estherk15 estherk15 merged commit c9e4be1 into master Aug 8, 2025
15 checks passed
@estherk15 estherk15 deleted the esther/docs-11720-pci-dss-deprecate branch August 8, 2025 19:13
StefonSimmons pushed a commit that referenced this pull request Aug 12, 2025
@estherk15 @jinatdatadog
@drichards-87 @StefonSimmons
Deprecate PCI DSS for Logs and APM (#30919)
956f462 
* Deprecate PCI DSS docs

* Remove references to PCI

* Re-add PCI Compliance docs with updated links

* Update content/en/data_security/pci_compliance.md

Co-authored-by: Esther Kim <[email protected]>

* Update content/en/data_security/pci_compliance.md

* Readd the further reading links

* Remove PCI config shortcodes

* Apply suggestions from code review

Co-authored-by: DeForest Richards <[email protected]>

* Restore shortcode to fix build issue

---------

Co-authored-by: jinatdatadog <[email protected]>
Co-authored-by: DeForest Richards <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcaudan bcaudan bcaudan approved these changes

@drichards-87 drichards-87 drichards-87 approved these changes

@jinatdatadog jinatdatadog jinatdatadog approved these changes

Assignees
No one assigned
Labels
Architecture Everything related to the Doc backend Guide Content impacting a guide
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@estherk15 @bcaudan @drichards-87 @jinatdatadog