Merge pull request #817 from DFE-Digital/er-790-azure-refinements

Er 790 azure refinements
DFE-Digital · Sep 7, 2023 · 367eeda · 367eeda
2 parents c857078 + 01ce54e
commit 367eeda
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/tf-azure-deploy.yml b/.github/workflows/tf-azure-deploy.yml
@@ -32,14 +32,14 @@ env:
   ARM_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}
   ARM_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
   ARM_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }}
+  ARM_USE_OIDC: true
+  ARM_SKIP_PROVIDER_REGISTRATION: true
 
 jobs:
   terraform-plan:
     name: 'Terraform Plan'
     runs-on: ubuntu-latest
     environment: ${{ inputs.environment }}
-    env:
-      ARM_SKIP_PROVIDER_REGISTRATION: true
     outputs:
       tfplanExitCode: ${{ steps.tf-plan.outputs.exitcode }}
 

diff --git a/terraform-azure/main.tf b/terraform-azure/main.tf
@@ -1,5 +1,6 @@
 provider "azurerm" {
-  use_oidc = true
+  use_oidc                   = true
+  skip_provider_registration = true
 
   features {
     resource_group {