Skip to content

Commit

Permalink
ci: remove unused subscriptionid for deployment
Browse files Browse the repository at this point in the history
  • Loading branch information
arealmaas committed Feb 7, 2024
1 parent 07d1983 commit baff594
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .azure/modules/containerAppJob/main.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ resource job 'Microsoft.App/jobs@2023-05-01' = {
secrets: [
{
// todo: move this and refactor into adding this somewhere else
name: 'adoconnectionstringsecreturi'
name: 'dialogdbconnectionstringsecreturi'
keyVaultUrl: adoConnectionStringSecretUri
identity: 'System'
}
Expand All @@ -36,7 +36,7 @@ resource job 'Microsoft.App/jobs@2023-05-01' = {
env: [
{
name: 'Infrastructure__DialogDbConnectionString'
secretRef: 'adoconnectionstringsecreturi'
secretRef: 'dialogdbconnectionstringsecreturi'
}
]
image: image
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/action-deploy-apps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,6 @@ jobs:
scope: resourcegroup
template: ./.azure/applications/web-api-migration-job/main.bicep
resourceGroupName: ${{ secrets.AZURE_RESOURCE_GROUP_NAME }}
# not needed becauase we sign in with the azure/login action using the subscriptionId
subscriptionId: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
deploymentMode: Incremental
deploymentName: "dp-be-${{ inputs.environment }}-web-api-migration-job-${{ inputs.gitShortSha }}"
region: ${{ inputs.region }}
Expand Down Expand Up @@ -151,8 +149,6 @@ jobs:
scope: resourcegroup
template: ./.azure/applications/${{ matrix.name }}/main.bicep
resourceGroupName: ${{ secrets.AZURE_RESOURCE_GROUP_NAME }}
# not needed becauase we sign in with the azure/login action using the subscriptionId
subscriptionId: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
deploymentMode: Incremental
deploymentName: dp-be-${{ inputs.environment }}-${{ matrix.name }}-${{ inputs.gitShortSha }}
region: ${{ inputs.region }}
Expand Down

0 comments on commit baff594

Please sign in to comment.