From abfdcc8cdbc8c4d6dccdbfdcac54270f7d83ed18 Mon Sep 17 00:00:00 2001
From: Yishuai Li <liyishuai@users.noreply.github.com>
Date: Thu, 17 Feb 2022 14:56:21 -0500
Subject: [PATCH] ci: release docker in source only, take 2 (#9132)

* ci: release docker in source only (#8971)

* ci: release docker in source only

* condition at job level

* condition over secrets

* Apply suggestions from code review

Co-authored-by: Tony <TonyRL@users.noreply.github.com>

Co-authored-by: Tony <TonyRL@users.noreply.github.com>

* ci: fix job condition

Co-authored-by: Tony <TonyRL@users.noreply.github.com>
---
 .github/workflows/docker-release.yml | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/.github/workflows/docker-release.yml b/.github/workflows/docker-release.yml
index ad05e4ea82..9862675157 100644
--- a/.github/workflows/docker-release.yml
+++ b/.github/workflows/docker-release.yml
@@ -12,8 +12,20 @@ on:
       - 'yarn.lock'
 
 jobs:
+  check-env:
+    runs-on: ubuntu-latest
+    outputs:
+      check-docker: ${{ steps.check-docker.outputs.defined }}
+    steps:
+      - id: check-docker
+        env:
+          DOCKER_USERNAME: ${{ secrets.DOCKER_USERNAME }}
+        if: ${{ env.DOCKER_USERNAME != '' }}
+        run: echo "::set-output name=defined::true"
   release:
     runs-on: ubuntu-latest
+    needs: check-env
+    if: needs.check-env.outputs.check-docker == 'true'
     steps:
       - name: Checkout
         uses: actions/checkout@v2