Skip to content

Commit

Permalink
Merge pull request #318 from DFE-Digital/bugfix/fhb-727-sql-firewall-…
Browse files Browse the repository at this point in the history
…name

chore(pipeline): Add unique SQL firewall name
  • Loading branch information
stevesatdfe authored Dec 16, 2024
2 parents 43c5ec5 + d431ffa commit 6bb3ef2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/run-sql-script.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ jobs:
ADMIN_KEY_VAULT_NAME: ${{ vars.AZURE_RESOURCE_PREFIX }}-kv-fh-admin
RESOURCE_GROUP: ${{ vars.AZURE_RESOURCE_PREFIX }}-familyhubs
SQL_SERVER: ${{ vars.AZURE_RESOURCE_PREFIX }}-as-fh-sql-server
SQL_FIREWALL_NAME: '"SQL ${{ inputs.display_name }} Rule"'
steps:
- name: Checkout Repository
uses: actions/checkout@v4
Expand All @@ -54,7 +55,7 @@ jobs:
action: "Add"
az_resource_group: ${{ env.RESOURCE_GROUP }}
az_sql_server_name: ${{ env.SQL_SERVER }}
az_firewall_rule_name: sql-script-rule
az_firewall_rule_name: ${{ env.SQL_FIREWALL_NAME }}

- name: Run SQL Script PowerShell
uses: ./.github/actions/run-sql-script
Expand All @@ -71,4 +72,4 @@ jobs:
action: "Remove"
az_resource_group: ${{ env.RESOURCE_GROUP }}
az_sql_server_name: ${{ env.SQL_SERVER }}
az_firewall_rule_name: sql-script-rule
az_firewall_rule_name: ${{ env.SQL_FIREWALL_NAME }}

0 comments on commit 6bb3ef2

Please sign in to comment.