Merge pull request #927 from dolthub/db/fix-ci-for-outside-contributors

/.github/workflows/ci-bats-tests.yaml: skip aws tests if no secrets found
dolthub · Oct 12, 2020 · 2327c85 · 2327c85
2 parents a7ca633 + 579927b
commit 2327c85
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/.github/workflows/ci-bats-tests.yaml b/.github/workflows/ci-bats-tests.yaml
@@ -12,8 +12,20 @@ jobs:
       fail-fast: true
       matrix:
         os: [ ubuntu-latest, macos-latest ]
+    env:
+      use_credentials: ${{ secrets.AWS_SECRET_ACCESS_KEY != '' && secrets.AWS_ACCESS_KEY_ID != '' }}
     steps:
+      - name: Conditionally Set ENV VARS for AWS tests
+        run: |
+          if [[ $use_credentials == true ]]; then
+            echo "AWS_SDK_LOAD_CONFIG=1" >> $GITHUB_ENV
+            echo "AWS_REGION=us-west-2" >> $GITHUB_ENV
+            echo "DOLT_BATS_AWS_TABLE=dolt-ci-bats-manifests-us-west-2" >> $GITHUB_ENV
+            echo "DOLT_BATS_AWS_BUCKET=dolt-ci-bats-chunks-us-west-2" >> $GITHUB_ENV
+            echo "DOLT_BATS_AWS_EXISTING_REPO=aws_remote_bats_tests" >> $GITHUB_ENV
+          fi
       - name: Configure AWS Credentials
+        if: ${{ env.use_credentials == 'true' }}
         uses: aws-actions/configure-aws-credentials@v1
         with:
           aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
@@ -56,8 +68,8 @@ jobs:
           go build -mod=readonly -o ../.ci_bin/remotesrv ./utils/remotesrv/.
       - name: Setup Dolt Config
         run: |
-          dolt config --global --add user.name 'Liquidata Actions'
-          dolt config --global --add user.email 'actions@liquidata.co'
+          dolt config --global --add user.name 'Dolthub Actions'
+          dolt config --global --add user.email 'actions@dolthub.com'
       - name: Install expect
         if: matrix.os == 'ubuntu-latest'
         run: sudo apt-get install -y expect
@@ -67,9 +79,3 @@ jobs:
         run: |
           bats --tap .
         working-directory: ./bats
-        env:
-          AWS_SDK_LOAD_CONFIG: "1"
-          AWS_REGION: "us-west-2"
-          DOLT_BATS_AWS_TABLE: "dolt-ci-bats-manifests-us-west-2"
-          DOLT_BATS_AWS_BUCKET: "dolt-ci-bats-chunks-us-west-2"
-          DOLT_BATS_AWS_EXISTING_REPO: "aws_remote_bats_tests"