Skip to content

Commit cb1359f

Browse files
committed
Integration: Remove explicit version passing to azure/CLI.
In the Windows CI's periodic runs the `azure/CLI` GitHub action library is leveraged to run various Azure-related commands. To avoid possible desyncing between the auth libraries of `azure/Login` and `azure/CLI` as described [here](Azure/cli#56 (comment)), this patch neglects to pass an explicit `azcliversion` to `azure/CLI` in order to have it default to the Azure CLI version set up by `azure/Login`. Signed-off-by: Nashwan Azhari <[email protected]>
1 parent db9c6fc commit cb1359f

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

.github/workflows/windows-periodic.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,28 +66,24 @@ jobs:
6666
- name: AZResourceGroupCreate
6767
uses: azure/CLI@v1
6868
with:
69-
azcliversion: 2.29.2
7069
inlinescript: |
7170
az group create -n ${{ matrix.AZURE_RESOURCE_GROUP }} -l ${{ env.AZURE_DEFAULT_LOCATION }} --tags creationTimestamp=$(date +%Y-%m-%dT%T%z)
7271
7372
- name: AZTestVMCreate
7473
uses: azure/CLI@v1
7574
with:
76-
azcliversion: 2.29.2
7775
inlinescript: |
7876
az vm create -n winTestVM --admin-username ${{ env.DEFAULT_ADMIN_USERNAME }} --admin-password ${{ env.PASSWORD }} --image ${{ matrix.AZURE_IMG }} -g ${{ matrix.AZURE_RESOURCE_GROUP }} --nsg-rule SSH --size ${{ env.AZURE_DEFAULT_VM_SIZE }}
7977
8078
- name: GetAZVMPublicIP
8179
uses: azure/CLI@v1
8280
with:
83-
azcliversion: 2.29.2
8481
inlinescript: |
8582
echo "VM_PUB_IP=$(az network public-ip list -g ${{ matrix.AZURE_RESOURCE_GROUP }} | jq '.[0]["ipAddress"]' | tr -d '\"')" >> $GITHUB_ENV
8683
8784
- name: EnableAZVMSSH
8885
uses: azure/CLI@v1
8986
with:
90-
azcliversion: 2.29.2
9187
inlinescript: |
9288
az vm run-command invoke --command-id RunPowerShellScript -n winTestVM -g ${{ matrix.AZURE_RESOURCE_GROUP }} --scripts @$GITHUB_WORKSPACE/script/setup/enable_ssh_windows.ps1 --parameters 'SSHPublicKey=${{ env.SSH_PUB_KEY }}'
9389
@@ -205,6 +201,5 @@ jobs:
205201
- name: ResourceCleanup
206202
uses: azure/CLI@v1
207203
with:
208-
azcliversion: 2.29.2
209204
inlinescript: |
210205
az group delete -g ${{ matrix.AZURE_RESOURCE_GROUP }} --yes

0 commit comments

Comments
 (0)