Skip to content

Commit

Permalink
ci: Force Subscription Load During Login
Browse files Browse the repository at this point in the history
  • Loading branch information
ellismg committed Nov 7, 2024
1 parent a805072 commit e7ffa92
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion cli/azd/cmd/auth_login.go
Original file line number Diff line number Diff line change
Expand Up @@ -338,7 +338,7 @@ func (la *loginAction) Run(ctx context.Context) (*actions.ActionResult, error) {
return nil, err
}

if la.flags.clientID == "" {
if v, err := strconv.ParseBool(os.Getenv("AZD_DEBUG_LOGIN_FORCE_SUBSCRIPTION_REFRESH")); (err == nil && v) || la.flags.clientID == "" {

Check failure on line 341 in cli/azd/cmd/auth_login.go

View workflow job for this annotation

GitHub Actions / azd-lint (ubuntu-latest)

the line is 139 characters long, which exceeds the maximum of 125 characters. (lll)
// Update the subscriptions cache for regular users (i.e. non-service-principals).
// The caching is done here to increase responsiveness of listing subscriptions in the application.
// It also allows an implicit command for the user to refresh cached subscriptions.
Expand Down
1 change: 1 addition & 0 deletions eng/pipelines/templates/jobs/build-cli.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ jobs:
AZURESUBSCRIPTION_TENANT_ID: $(AzureSubscriptionTenantId)
AZURESUBSCRIPTION_SERVICE_CONNECTION_ID: $(AzureSubscriptionServiceConnectionId)
SYSTEM_ACCESSTOKEN: $(System.AccessToken)
AZD_DEBUG_LOGIN_FORCE_SUBSCRIPTION_REFRESH: true
displayName: AZD Login
- task: AzureCLI@2
Expand Down

0 comments on commit e7ffa92

Please sign in to comment.