Add dev cluster env to ga_wif naming #355
Merged
+1
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
ga_wif naming doesn't include the dev cluster env, so cluster build fails if one already exists
Error: A resource with the ID "/subscriptions/nnnnnnn/resourceGroups/s189d01-tsc-dv-rg/providers/Microsoft.ManagedIdentity/userAssignedIdentities/s189d01-ga-wif-development-bat-id" already exists - to be managed via Terraform this resource needs to be imported into the State. Please see the resource documentation for "azurerm_user_assigned_identity" for more information.
Changes proposed in this pull request
use var.environment instead of var.config
this will create s189d01-ga-wif-clustern-bat-id instead of s189d01-ga-wif-development-bat-id
Guidance to review
see dev cluster 1
make env deploy-plan makes no changes to test, production clusters
Checklist
Devops
label