Skip to content

Commit 22a2467

Browse files
Refactor Azure authentication conditions in action YAML files
1 parent 507af1f commit 22a2467

5 files changed

Lines changed: 10 additions & 10 deletions

File tree

tasks/dbt-docs-publish/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,13 +67,13 @@ runs:
6767
6868
- name: Authenticate to Azure as a Service Principal
6969
uses: azure/login@v2
70-
condition: ${{ inputs.use-azure-federated-login == 'false' }}
70+
if: ${{ inputs.use-azure-federated-login == 'false' }}
7171
with:
7272
creds: ${{ inputs.azure-credentials }}
7373

7474
- name: Authenticate to Azure as a Service Principal (ODIC)
7575
uses: azure/login@v2
76-
condition: ${{ inputs.use-azure-federated-login == 'true' }}
76+
if: ${{ inputs.use-azure-federated-login == 'true' }}
7777
with:
7878
client-id: ${{ inputs.azure-client-id }}
7979
tenant-id: ${{ inputs.azure-tenant-id }}

tasks/dbt-manifest-clone/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,13 @@ runs:
3737
steps:
3838
- name: Authenticate to Azure as a Service Principal
3939
uses: azure/login@v2
40-
condition: ${{ inputs.use-azure-federated-login == 'false' }}
40+
if: ${{ inputs.use-azure-federated-login == 'false' }}
4141
with:
4242
creds: ${{ inputs.azure-credentials }}
4343

4444
- name: Authenticate to Azure as a Service Principal (ODIC)
4545
uses: azure/login@v2
46-
condition: ${{ inputs.use-azure-federated-login == 'true' }}
46+
if: ${{ inputs.use-azure-federated-login == 'true' }}
4747
with:
4848
client-id: ${{ inputs.azure-client-id }}
4949
tenant-id: ${{ inputs.azure-tenant-id }}

tasks/dbt-manifest-destroy/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@ runs:
3434
steps:
3535
- name: Authenticate to Azure as a Service Principal
3636
uses: azure/login@v2
37-
condition: ${{ inputs.use-azure-federated-login == 'false' }}
37+
if: ${{ inputs.use-azure-federated-login == 'false' }}
3838
with:
3939
creds: ${{ inputs.azure-credentials }}
4040

4141
- name: Authenticate to Azure as a Service Principal (ODIC)
4242
uses: azure/login@v2
43-
condition: ${{ inputs.use-azure-federated-login == 'true' }}
43+
if: ${{ inputs.use-azure-federated-login == 'true' }}
4444
with:
4545
client-id: ${{ inputs.azure-client-id }}
4646
tenant-id: ${{ inputs.azure-tenant-id }}

tasks/dbt-manifest-download/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,13 @@ runs:
4040
steps:
4141
- name: Authenticate to Azure as a Service Principal
4242
uses: azure/login@v2
43-
condition: ${{ inputs.use-azure-federated-login == 'false' }}
43+
if: ${{ inputs.use-azure-federated-login == 'false' }}
4444
with:
4545
creds: ${{ inputs.azure-credentials }}
4646

4747
- name: Authenticate to Azure as a Service Principal (ODIC)
4848
uses: azure/login@v2
49-
condition: ${{ inputs.use-azure-federated-login == 'true' }}
49+
if: ${{ inputs.use-azure-federated-login == 'true' }}
5050
with:
5151
client-id: ${{ inputs.azure-client-id }}
5252
tenant-id: ${{ inputs.azure-tenant-id }}

tasks/dbt-manifest-upload/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,13 +41,13 @@ runs:
4141
steps:
4242
- name: Authenticate to Azure as a Service Principal
4343
uses: azure/login@v2
44-
condition: ${{ inputs.use-azure-federated-login == 'false' }}
44+
if: ${{ inputs.use-azure-federated-login == 'false' }}
4545
with:
4646
creds: ${{ inputs.azure-credentials }}
4747

4848
- name: Authenticate to Azure as a Service Principal (ODIC)
4949
uses: azure/login@v2
50-
condition: ${{ inputs.use-azure-federated-login == 'true' }}
50+
if: ${{ inputs.use-azure-federated-login == 'true' }}
5151
with:
5252
client-id: ${{ inputs.azure-client-id }}
5353
tenant-id: ${{ inputs.azure-tenant-id }}

0 commit comments

Comments
 (0)