Skip to content

Commit

Permalink
fix choosing cluster logic
Browse files Browse the repository at this point in the history
  • Loading branch information
AlisonB319 committed Mar 20, 2024
1 parent 3587331 commit d3cc05d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions e2e/cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -320,9 +320,9 @@ func chooseCluster(
var chosenConfig clusterConfig
for i := range clusterConfigs {
config := &clusterConfigs[i]
if scenario.Airgap && !config.isAirgapCluster {
if (scenario.Airgap && !config.isAirgapCluster) || (config.isAirgapCluster && !scenario.Airgap) {
continue
}
}

if scenario.Config.ClusterSelector(config.cluster) {
// only validate + prep the cluster for testing if we didn't just create it and it hasn't already been prepared
Expand Down
2 changes: 1 addition & 1 deletion e2e/vmss.go
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ func (r VMSSOperationRetrier) createVMSSWithPayload(ctx context.Context, customD
if pollErr == nil {
return &vmssResp.VirtualMachineScaleSet, nil
} else {
log.Printf("failed to create VMSS %q: %v. Retry attempts left: %d", vmssName, pollErr, r.maxRetries-(i+1))
log.Printf("failed to create VMSS. Retry attempts left: %d", r.maxRetries-(i+1))
}
}
return nil, fmt.Errorf("unable to create VMSS %q: %w", vmssName, pollErr)
Expand Down

0 comments on commit d3cc05d

Please sign in to comment.