Merge pull request #775 from techmatters/CHI-3008-fix

Fixing the control flow for security group param in Generate training set script
techmatters · Nov 6, 2024 · 1a566a3 · 1a566a3
2 parents 478c865 + aeafdae
commit 1a566a3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/generate-ai-training-ecs.yml b/.github/workflows/generate-ai-training-ecs.yml
@@ -53,8 +53,8 @@ jobs:
           CLUSTER="${{ inputs.environment}}-ecs-cluster"
           FAMILY="${{ inputs.environment}}-hrm-scheduled-task"
           SUBNETS=$(if [ ${{ inputs.environment}} == "development" ]; then echo "subnet-034e5c652dbad09dd"; else echo "subnet-00b4e88c0ea178f3d"; fi)
-          SECURITY_GROUPS=(if [ ${{ inputs.environment}} == "development" ]; then echo ("sg-09194f9a648baf082" "sg-047498d4c7b2cedd8" "sg-06d6458accc0ec5ed" "sg-0ace4338c75e5d3de") 
-            else echo ("sg-02fd053fe7b4660ed" "sg-0e7119c4423f2c7b0" "sg-066d0055caa7cbe90" "sg-0c7fb74e11cb64ff8") fi)
+          SECURITY_GROUPS=($(if [ ${{ inputs.environment}} == "development" ]; then echo "sg-09194f9a648baf082 sg-047498d4c7b2cedd8 sg-06d6458accc0ec5ed sg-0ace4338c75e5d3de"; 
+            else echo "sg-02fd053fe7b4660ed sg-0e7119c4423f2c7b0 sg-066d0055caa7cbe90 sg-0c7fb74e11cb64ff8"; fi))
           COMMAND="npm run start:generate-ai-training-set ${{ inputs.environment }} ${{ inputs.helpline-shortcodes }} tl-aselo-ai-${{ inputs.environment}}"
           TASK_ROLE="arn:aws:iam::${{ secrets.AWS_ACCOUNT_ID }}:role/${{ inputs.environment}}-ecsTaskRole"
           EXEC_ROLE="arn:aws:iam::${{ secrets.AWS_ACCOUNT_ID }}:role/${{ inputs.environment}}-ecsTaskExecutionRole"