From 59417535df49d8a61a91c02112ea66a2ec854f48 Mon Sep 17 00:00:00 2001 From: Matthew Garrett Date: Thu, 24 Aug 2023 09:27:31 -0700 Subject: [PATCH] Identified issue, fixed two identified code bugs --- .github/workflows/azure-ipam-testing.yml | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/.github/workflows/azure-ipam-testing.yml b/.github/workflows/azure-ipam-testing.yml index 3c2aa9a..d7f95dc 100644 --- a/.github/workflows/azure-ipam-testing.yml +++ b/.github/workflows/azure-ipam-testing.yml @@ -80,15 +80,7 @@ jobs: $composeYaml['services']['ipam-engine'].image = $engineContainer $composeYaml['services']['nginx-proxy'].image = $lbContainer - $composeYaml | ConvertTo-Yaml | Out-File -FilePath ./docker-compose-prod.yml - - Write-Host "-----------------------------------" - Get-ChildItem | Select-Object Name - Write-Host "-----------------------------------" - $composeYaml | ConvertTo-Yaml - Write-Host "-----------------------------------" - Get-Content -Path ./docker-compose.prod.yml - Write-Host "-----------------------------------" + $composeYaml | ConvertTo-Yaml | Out-File -FilePath ./docker-compose.prod.yml - name: Deploy Azure IPAM working-directory: deploy @@ -196,7 +188,7 @@ jobs: $engineApp = Get-AzADApplication -ApplicationId ${{ needs.deploy.outputs.ipamEngineAppId }} Remove-AzResourceGroup -Name ${{ needs.deploy.outputs.ipamResourceGroup }} -Force - Remove-AzRoleAssignment -ObjectId $engineApp.ObjectId -Scope $scope -RoleDefinitionName Reader + Remove-AzRoleAssignment -ObjectId $engineApp.Id -Scope $scope -RoleDefinitionName Reader $uiApp | Remove-AzADApplication $engineApp | Remove-AzADApplication