Skip to content

Conversation

kennonkwok
Copy link
Contributor

What does this PR do? What is the motivation?

Adding a guide on managing the DatadogPodAutoscaler resources using Terraform. This has been a frequent topic in conversations with customers.

Merge instructions

No special instructions.

Merge readiness:

  • Ready for merge

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

@kennonkwok kennonkwok requested a review from a team as a code owner August 21, 2025 19:21
@github-actions github-actions bot added the Guide Content impacting a guide label Aug 21, 2025
Copy link
Contributor

github-actions bot commented Aug 21, 2025

📝 Documentation Team Review Required

This pull request requires approval from the @DataDog/documentation team before it can be merged.

Please ensure your changes follow our documentation guidelines and wait for a team member to review and approve your changes.

Copy link
Contributor

@drichards-87 drichards-87 added the editorial review Waiting on a more in-depth review label Aug 21, 2025
@drichards-87
Copy link
Contributor

Created DOCS-11868 for Docs Team review.

Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

Thanks for preparing this guide, @kennonkwok! I added some editorial suggestions throughout, and there are a few larger-scale changes that I think would help:

  • Streamline the heading structure so it's easier to follow the stages and their sub-steps.
  • Refactor code examples to use the code block shortcode, which includes the filename (so we wouldn't need to put filenames in headings)
  • Refactor the Additional Resources section into a Further Reading shortcode section

Those changes are hard to convey with GitHub line-edit comments, so I can pull down your branch and add them in separate commits.

Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

Per 52c0118, there are several other instances of DKA that probably also need to be changed to DPA, including code examples and file/folder names.

@kennonkwok
Copy link
Contributor Author

Per 52c0118, there are several other instances of DKA that probably also need to be changed to DPA, including code examples and file/folder names.

I can take a pass over those.

@kennonkwok
Copy link
Contributor Author

Per 52c0118, there are several other instances of DKA that probably also need to be changed to DPA, including code examples and file/folder names.

OK, I think it's good now, I updated the overview to mention both the DKA and DPA for clarity since DKA is still mentioned in the example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial review Waiting on a more in-depth review Guide Content impacting a guide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants