-
Notifications
You must be signed in to change notification settings - Fork 134
docs: friends don't let friends editorialize the quickstart guides #166
docs: friends don't let friends editorialize the quickstart guides #166
Conversation
@@ -1,30 +1,37 @@ | |||
# Quickstart: BKPR on Azure Kubernetes Service (AKS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we rename this to AKS Quickstart to be consistent with the README.md document that links here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not too keen on changing the title.
I would like to know what @arapulido and @anguslees thinks about this.
az ad app delete \ | ||
--subscription ${AZURE_SUBSCRIPTION_ID} \ | ||
--id $(jq -r .oauthProxy.client_id kubeprod-autogen.json) | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@anguslees @falfaro I've added these commands to clean up the app registrations. Ideally we should adding a kubeprod reset
command to automatically clean up these things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But I still think we should rename the title to "AKS Quickstart" to match what we have in README.md.
No description provided.