[WP] Use auto generated config doc - #39455
Conversation
This comment has been minimized.
This comment has been minimized.
Preview links (active after the
|
866f23d to
d535ca2
Compare
| 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 |
There was a problem hiding this comment.
we should remove this. the generated file from websites-sources is not committed to the repo
| content/en/security/workload_protection/workload_protection_agent_config.md |
There was a problem hiding this comment.
Maybe we should remove the other ones then since they are using the same mechanism. Maybe these lines are artifacts from the previous mechanism
There was a problem hiding this comment.
they are artifacts from previous mechanisms. you can remove them
| path: /security/workload_protection/workload_protection_agent_config | ||
| aliases: | ||
| - /security/workload_protection/setup/advanced_configuration | ||
| - /security/workload_protection/setup/agent_variables |
There was a problem hiding this comment.
@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.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
@theop-dd the left nav should open to Setup -> Advanced Configuration for your new page but it is stuck at the Workload Protection header
3 participants
What does this PR do? What is the motivation?
This PR use the auto generated version of the file
workload_protection/setup/advanced_configurationfrom the repodatadog-agentinstead of the hardcoded version. This PR is part of the global revamp of Workload Protection Documentation.Merge readiness
For Datadog employees:
<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./reviewto run an automated check that catches common issues before a Documentation team member reviews your PR.AI assistance
Additional notes