Skip to content
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.

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

docs: Provide walkthrough for managed identity #1578

Merged

Conversation

Mimetis
Copy link
Contributor

@Mimetis Mimetis commented Apr 1, 2021

Docs: Adding a walkthrough to deploy Promitor with Managed Identity (supported by AAD Pod Identity)

@Mimetis Mimetis requested a review from tomkerkhove as a code owner April 1, 2021 12:02
@tomkerkhove tomkerkhove changed the base branch from master to mi-walkthrough April 1, 2021 15:07
@tomkerkhove tomkerkhove changed the title docs: Mimetis/docs managedidentity docs: Provide walkthrough for managed identity Apr 1, 2021
Copy link
Owner

@tomkerkhove tomkerkhove left a comment

Choose a reason for hiding this comment

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

Thanks! I'll merge it in a temp branch and tweak the wording a bit but is super helpful!

@tomkerkhove tomkerkhove merged commit 5a77f6b into tomkerkhove:mi-walkthrough Apr 1, 2021
tomkerkhove added a commit that referenced this pull request Apr 5, 2021
* docs: Provide walkthrough for managed identity (#1578)

Co-authored-by: Tom Kerkhove <[email protected]>

* Rename walkthrough

Signed-off-by: Tom Kerkhove <[email protected]>

* Reshuffle intro

Signed-off-by: Tom Kerkhove <[email protected]>

* Change title

Signed-off-by: Tom Kerkhove <[email protected]>

* Current TOC

Signed-off-by: Tom Kerkhove <[email protected]>

* Create & auth to cluster

Signed-off-by: Tom Kerkhove <[email protected]>

* Deprecate feature

Signed-off-by: Tom Kerkhove <[email protected]>

* Finalize

Signed-off-by: Tom Kerkhove <[email protected]>

* System assigned

Signed-off-by: Tom Kerkhove <[email protected]>

* Linting

Signed-off-by: Tom Kerkhove <[email protected]>

Co-authored-by: Sébastien Pertus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants