fix: script permissions and wiring in pipelines

dash0hq · Nov 7, 2023 · bcf8b79 · bcf8b79
1 parent 9f52df3
commit bcf8b79
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -83,7 +83,7 @@ jobs:
         # is available only for pull trigger. Otherwise, use the branch on which the commit was pushed.
         # (Ref names in pull requests are in the shape of `<pr_id>/merge` and those will break CDK.)
         run: |
-          ./.github/workflows/scripts/test_env_name.sh >> $GITHUB_OUTPUT || exit 1
+          echo "test_env_name=$(./.github/workflows/scripts/test_env_name.sh)" >> $GITHUB_OUTPUT || exit 1
 
       - name: Deploy validation backend
         shell: bash

diff --git a/.github/workflows/clean-up-test-env.yaml b/.github/workflows/clean-up-test-env.yaml
@@ -35,7 +35,7 @@ jobs:
         # is available only for pull trigger. Otherwise, use the branch on which the commit was pushed.
         # (Ref names in pull requests are in the shape of `<pr_id>/merge` and those will break CDK.)
         run: |
-          ./.github/workflows/scripts/test_env_name.sh >> $GITHUB_OUTPUT || exit 1
+          echo "test_env_name=$(./.github/workflows/scripts/test_env_name.sh)" >> $GITHUB_OUTPUT || exit 1
 
       - name: Delete validation backend
         shell: bash

diff --git a/.github/workflows/scripts/test_env_name.sh b/.github/workflows/scripts/test_env_name.sh
@@ -1,4 +1,4 @@
-#!/bin/env bash
+#!/usr/bin/env bash
 
 # Ensure test name is not long enough to break restrictions on length names in AWS (e.g., role-name length)
 # This logic ensures test envs have max length 18. Names of length 13 to 17 will become of length 18 to ensure