Skip to content

[WP] Use auto generated config doc #39455

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

Open
theop-dd wants to merge 2 commits into
base: master
Choose a base branch
from
+7 −54
Open

[WP] Use auto generated config doc #39455

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d535ca2
[WP] Use auto generated config doc
theop-dd Aug 25, 2026
1edf40b
Update URL for Advanced Configuration in workload protection menu
StefonSimmons Sep 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions hugo/.gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,7 @@ content/en/security/workload_protection/backend_linux.md
content/en/security/workload_protection/backend_windows.md
content/en/security/workload_protection/linux_expressions.md
content/en/security/workload_protection/windows_expressions.md
content/en/security/workload_protection/workload_protection_agent_config.md

@StefonSimmons StefonSimmons Sep 10, 2026 •
edited
Loading

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.

we should remove this. the generated file from websites-sources is not committed to the repo

Suggested change
content/en/security/workload_protection/workload_protection_agent_config.md

@theop-dd theop-dd Sep 10, 2026

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.

Maybe we should remove the other ones then since they are using the same mechanism. Maybe these lines are artifacts from the previous mechanism

@StefonSimmons StefonSimmons Sep 11, 2026 •
edited
Loading

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.

they are artifacts from previous mechanisms. you can remove them


# security threat detection
content/en/security/threats/backend_linux.md
Expand Down
2 changes: 1 addition & 1 deletion hugo/config/_default/menus/main.en.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8676,7 +8676,7 @@ menu:
identifier: workload_protection_setup_windows
weight: 70
- name: Advanced Configuration
url: security/workload_protection/setup/advanced_configuration
url: security/workload_protection/workload_protection_agent_config
parent: workload_protection_setup
identifier: workload_protection_setup_advanced_configuration
weight: 80
Expand Down
5 changes: 5 additions & 0 deletions hugo/content/en/security/workload_protection/_index.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,11 @@ cascade:
path: /security/workload_protection/windows_expressions
aliases:
- /security/threats/windows_expressions
- _target:
path: /security/workload_protection/workload_protection_agent_config
aliases:
- /security/workload_protection/setup/advanced_configuration
- /security/workload_protection/setup/agent_variables

@StefonSimmons StefonSimmons Sep 10, 2026

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.

@theop-dd So, I dont think this resulted in the expected outcome.

websites-sources#688 generates a new Docs file at content/en/security/workload_protection/workload_protection_agent_config.md. however, the following path doesnt exist for the left nav:

This Docs PR wants to remove the setup/advanced_configuration.md file and replace it with the above generated file, correct? If so, the left nav needs to reflect the change as well.

@theop-dd theop-dd Sep 10, 2026

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.

I can see the page I wanted to generate here too: https://docs-staging.datadoghq.com/theop-dd/add-auto-generated-config-WP/security/workload_protection/setup/advanced_configuration so it looks like it's working.
I'm not sure to understand the issue exactly, we can see it in the nav, it's under Setup/Advanced_Configuration which is the expected place

@StefonSimmons StefonSimmons Sep 11, 2026

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.

@theop-dd the left nav should open to Setup -> Advanced Configuration for your new page but it is stuck at the Workload Protection header

@StefonSimmons StefonSimmons Sep 11, 2026

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.

Screenshot 2026-09-11 at 3 15 24 PM


---

Expand Down
53 changes: 0 additions & 53 deletions hugo/content/en/security/workload_protection/setup/advanced_configuration.md
View file
Open in desktop

This file was deleted.

Loading