Skip to content

remove managedIdentityCredential call - #633

Closed
pl-jankowskimichal wants to merge 1 commit into
v1.18-ddfrom
pl-jankowskimichal-oidc-auth
Closed

remove managedIdentityCredential call#633
pl-jankowskimichal wants to merge 1 commit into
v1.18-ddfrom
pl-jankowskimichal-oidc-auth

Conversation

@pl-jankowskimichal

@pl-jankowskimichal pl-jankowskimichal commented Oct 27, 2025

Copy link
Copy Markdown
Member

just trying out an idea here. not a proposal at this point..

I actually have no idea if this works but I tried setting some extra env vars like:

 {
      "name": "AZURE_TOKEN_CREDENTIALS",
      "value": "WorkloadIdentityCredential"
  },
  {
      "name": "AZURE_AUTHORITY_HOST",
      "value": "https://login.microsoftonline.com/"
  },
 

with and without setting --azure-user-assigned-identity-id to ""
and I'm still getting the same error

{"error":"azure.BearerAuthorizer#WithAuthorization: Failed to refresh the Token for request to https://management.azure.com/subscriptions/<REDACTED>/providers/Microsoft.Network/virtualNetworks?api-version=2021-08-01: StatusCode=400 -- Original Error: adal: Refresh request failed. Status Code = '400'. Response body: {\"error\":\"invalid_request\",\"error_description\":\"Identity not found\"} Endpoint http://169.254.169.254/metadata/identity/oauth2/token?api-version=2018-02-01\u0026client_id=%22%22\u0026resource=https%3A%2F%2Fmanagement.core.windows.net%2F","level":"warning","msg":"Unable to synchronize Azure virtualnetworks list","subsys":"azure","time":"2025-10-23T08:17:10.926229744Z"}

adal is an old azure library
I'm still figuring out what is going on here

@pl-jankowskimichal

pl-jankowskimichal commented Oct 28, 2025

Copy link
Copy Markdown
Member Author

the deployment system was picking an older image (1.17)

"AZURE_TOKEN_CREDENTIALS" was not needed, setting AZURE_AUTHORITY_HOST seems to be enough. so any code changes are not needed 🎉

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.

1 participant