Skip to content

Commit 7720140

Browse files
fix: quota check script with dynamic region handling (#381)
* Made region list dynamic using variables for better flexibility and scalability. * comma separated values
1 parent 4541954 commit 7720140

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.github/workflows/RAdeploy.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ jobs:
2525
export AZURE_SUBSCRIPTION_ID="${{ secrets.AZURE_SUBSCRIPTION_ID }}"
2626
export GPT_MIN_CAPACITY="30"
2727
export TEXT_EMBEDDING_MIN_CAPACITY="45"
28+
export AZURE_REGIONS="${{ vars.AZURE_REGIONS_RA }}"
2829
2930
chmod +x ResearchAssistant/Deployment/scripts/checkquota.sh
3031
if ! ResearchAssistant/Deployment/scripts/checkquota.sh; then

ResearchAssistant/Deployment/scripts/checkquota.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/bin/bash
22

33
# List of Azure regions to check for quota (update as needed)
4-
REGIONS=("eastus2" "westus" "northcentralus" "uksouth" "francecentral")
4+
IFS=', ' read -ra REGIONS <<< "$AZURE_REGIONS"
55

66
SUBSCRIPTION_ID="${AZURE_SUBSCRIPTION_ID}"
77
GPT_MIN_CAPACITY="${GPT_MIN_CAPACITY}"
@@ -18,7 +18,7 @@ if ! az login --service-principal -u "$AZURE_CLIENT_ID" -p "$AZURE_CLIENT_SECRET
1818
fi
1919

2020
echo "🔄 Validating required environment variables..."
21-
if [[ -z "$SUBSCRIPTION_ID" || -z "$GPT_MIN_CAPACITY" || -z "$TEXT_EMBEDDING_MIN_CAPACITY" ]]; then
21+
if [[ -z "$SUBSCRIPTION_ID" || -z "$GPT_MIN_CAPACITY" || -z "$TEXT_EMBEDDING_MIN_CAPACITY" || -z "$REGIONS" ]]; then
2222
echo "❌ ERROR: Missing required environment variables."
2323
exit 1
2424
fi

0 commit comments

Comments
 (0)