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: Update documentation to reflect change from 'tenantId' to 'tenantID' #1057
Conversation
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.
Thanks for the PR @reetasingh!
There are few other places in the website
directory this needs to be updated.
grep -rnw website/ -e 'tenantId'
website/content/en/configurations/custom-environments.md:58:If the target cloud's identity provider system is [AD FS][adfs] (instead of Azure AD), then the `tenantId` property in `SecretProviderClass` should be set to `"adfs"`.
website/content/en/configurations/custom-environments.md:65: tenantId: "adfs"
website/content/en/configurations/identity-access-modes/pod-identity-mode.md:36: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/identity-access-modes/user-assigned-msi-mode.md:38: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/identity-access-modes/service-principal-mode.md:36: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/identity-access-modes/workload-identity-mode.md:37: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/identity-access-modes/system-assigned-msi-mode.md:38: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/sync-multiple-versions.md:38: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/ingress-tls.md:72:* Set the `tenantId` and `keyvaultName`
website/content/en/configurations/ingress-tls.md:105: tenantId: $TENANT_ID # the tenant ID of the KeyVault
website/content/en/configurations/sync-with-k8s-secrets.md:44: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/configurations/set-env-var.md:44: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/demos/standard-walkthrough/_index.md:106: tenantId: "${TENANT_ID}"
website/content/en/getting-started/usage/_index.md:62: tenantId: "tid" # the tenant ID of the KeyVault
website/content/en/getting-started/usage/_index.md:85: | tenantId | yes | tenant ID containing the Key Vault instance. Should be set to `"adfs"` for [Azure Stack Hub clouds](../../configurations/custom-environments) using the AD FS identity provider system
@aramase fixed it |
Codecov Report
@@ Coverage Diff @@
## master #1057 +/- ##
=======================================
Coverage 59.16% 59.16%
=======================================
Files 10 10
Lines 1097 1097
=======================================
Hits 649 649
Misses 413 413
Partials 35 35 |
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.
Thank you for the PR @reetasingh 🎉
lgtm
/azp run pr-e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
Reason for Change:
tenantId param will be deprecated in favor of tenantID. update the documentation to reflect change from 'tenantId' to 'tenantID
Requirements
Issue Fixed:
fixes #1027
Does this change contain code from or inspired by another project?
If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers: