[reorg fix] [DOCS-15112] Document Azure Secretless Auth setup and migration - #39073
[reorg fix] [DOCS-15112] Document Azure Secretless Auth setup and migration#39073jhgilbert wants to merge 11 commits into
Conversation
jhgilbert
added
WORK IN PROGRESS
github-actions
Bot
added
Images
Preview links (active after the
|
buraizu
added
editorial review
evazorro
left a comment
evazorro
left a comment
There was a problem hiding this comment.
Nothing blocking! A couple questions and small cosmetic things I noticed, but this is fine to merge as-is in my opinion.
|
|
||
| {{< img src="/getting_started/integrations/azure/GSwAzure_secretlessMigration.png" alt="The General tab for an app registration, with a callout recommending Secretless Auth and a Set Up Secretless Auth button" style="width:100%;" >}} | ||
|
|
||
| In the {{< ui >}}Secretless Authentication Setup{{< /ui >}} dialog, select a setup method, then complete the steps for that method: |
There was a problem hiding this comment.
Very minor, but it seems to me that this could be step 3 rather than body text following the procedure. Up to you, though!
There was a problem hiding this comment.
Ah -- I think it's probably because it introduces the three tabs that follow.
Definitely out of scope for this PR, but the structure on this page seems like it has grown a little complicated: tabs within collapse sections, etc. Could be worth revisiting at some point to see whether it makes sense to break it up.
|
|
||
| 1. Add the federated credential resource shown in the dialog to the same configuration that manages your Datadog Azure integration, then apply it. Copy the `issuer` and `subject` values from the dialog. | ||
|
|
||
| {{< code-block lang="hcl" filename="" disable_copy="false" collapsible="false" >}} |
There was a problem hiding this comment.
This code block looks like it might be misaligned with the step (or it could just be a quirk of the tab). Maybe 4 spaces instead of 3?
|
|
||
| ## Next steps | ||
|
|
||
| {{< site-region region="us,us3,us5,eu,ap1,ap2,uk1" >}} |
There was a problem hiding this comment.
This might be unavoidable with the site-region shortcode, but it looks like this divides the list:
3 participants
🤖 Auto-generated fix for #38328.
@buraizu — the docs repo reorg created merge conflicts in your PR #38328. This is an auto-generated replacement with the file paths fixed; please use it instead of the original.
This PR replays the commits from #38328 with file paths translated to the post-reorg
hugo/layout. The original commits are preserved — same messages and authorship.The original PR (#38328) will be closed in favor of this one.
Next steps:
WORK IN PROGRESSlabel from this PR.Original PR description:
What does this PR do? What is the motivation?
Fixes DOCS-15112
Documents Azure Secretless Auth (currently in Preview) on the Getting Started with Azure page and cross-links the migration flow from two Azure guides:
secretless_auth_enabled.Merge instructions
Draft / work in progress. Not ready for review or merge yet — see Additional notes.
Merge readiness:
Additional notes
Still to confirm before this is review-ready: