Details
-
Bug
-
Resolution: Fixed
-
Major
-
7.1.1
-
1
Description
What is the issue?
Currently the "username/password" Service Principal authentication method for Azure is missing the AZURE_TENANT_ID environment variable. Looking at the new Azure SDK code, this variable is required for all Service Principal authentication methods:
// NewEnvironmentCredential creates an EnvironmentCredential. Pass nil to accept default options.
|
func NewEnvironmentCredential(options *EnvironmentCredentialOptions) (*EnvironmentCredential, error) {
|
if options == nil {
|
options = &EnvironmentCredentialOptions{}
|
}
|
tenantID := os.Getenv("AZURE_TENANT_ID")
|
if tenantID == "" {
|
return nil, errors.New("missing environment variable AZURE_TENANT_ID")
|
}
|
|
...
|
but in the SDK library's documentation it is not listed: https://pkg.go.dev/github.com/Azure/azure-sdk-for-go/sdk/azidentity#section-readme
We need to update our documentation to reflect the real required environment variables and not the ones documented.
Attachments
For Gerrit Dashboard: DOC-10203 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
176720,3 | DOC-10203 Add missing required env variable for Azure Service Principal | neo | backup | Status: MERGED | +2 | +1 |
176791,1 | Merge branch 'neo' into master | master | backup | Status: MERGED | +2 | +1 |