Skip to content

Conversation

@git-thuerk-done
Copy link
Contributor

@git-thuerk-done git-thuerk-done commented Oct 27, 2025

What does this PR do? What is the motivation?

Breaks out the prerequisites section into 2 tabs:

  • AWS (no new content)
  • Azure (all new content)
  • Updates screenshots and videos throughout to reflect new Azure tab
  • Adds new copy to the Group by and group by tags section to reflect new functionality for labels and service tags

Merge 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

@git-thuerk-done git-thuerk-done requested a review from a team as a code owner October 27, 2025 22:15
@github-actions github-actions bot added the Images Images are added/removed with this PR label Oct 27, 2025
@git-thuerk-done git-thuerk-done added the WORK IN PROGRESS No review needed, it's a wip ;) label Oct 27, 2025
@github-actions
Copy link
Contributor

@jamesponddotco
Copy link
Contributor

From the Cloudcraft team, this looks good. I'd just update this part:

These diagrams visually represent AWS cloud resources, allowing you to explore and analyze your environments.

I'd update it to include Azure instead of mentioning AWS only.

@git-thuerk-done
Copy link
Contributor Author

AWS

Thanks @jamesponddotco good catch! Just merged that update.

Copy link
Contributor

@buraizu buraizu left a comment

Choose a reason for hiding this comment

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

Looks good overall, but left a few suggestions and posted some questions.

git-thuerk-done and others added 3 commits November 7, 2025 16:24
@git-thuerk-done
Copy link
Contributor Author

Thanks @buraizu ! I incorporated your feedback in these 2 commits:
One
Two
For the 2nd commit, you'll notice I also moved up the requirements for Azure to above the resource collection section

Copy link
Contributor

@buraizu buraizu 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 applying that feedback! Just reiterated a previous suggestion with some additional context, and I also posted another question about the updated Azure steps

Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>
@git-thuerk-done git-thuerk-done merged commit 3d0d9d8 into master Nov 12, 2025
21 of 23 checks passed
@git-thuerk-done git-thuerk-done deleted the aliciathuerk/DOCS-12260-Cloudcraft-Azure branch November 12, 2025 15:13
dussault-antoine pushed a commit that referenced this pull request Nov 13, 2025
* adding Azure support and updating screenshots

* updating permissions for Azure

* fix TOC snafu

* testing commit image

* add info on group by and labels

* adding back image

* updating infra and obsverability images

* update SDS image

* vuln and ccm

* small wording tweak

* feedback from James' review

* Apply suggestions from Bryce's review

Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>

* incorporating feedback

* fix spacing

* Apply suggestions from code review

Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>

* wording tweak

---------

Co-authored-by: aliciascott <alicia.scott@datadoghq.com>
Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.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 WORK IN PROGRESS No review needed, it's a wip ;)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants