Use default tag in case PR

In PRs on forks the secrets are not exposed and we need to default to something other than an empty string. Signed-off-by: Nicolas Bock <[email protected]>
canonical · May 6, 2024 · cf8a002 · cf8a002
1 parent 979326c
commit cf8a002
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/.github/workflows/docker-publish.yaml b/.github/workflows/docker-publish.yaml
@@ -13,6 +13,7 @@ on:
 env:
   REGISTRY: docker.io
   IMAGE_NAME: ${{ github.repository }}
+  DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME || 'unpublished' }}
 
 jobs:
   build:
@@ -58,7 +59,7 @@ jobs:
         id: processor_meta
         uses: docker/metadata-action@v5
         with:
-          images: ${{ secrets.DOCKERHUB_USERNAME }}/athena-processor
+          images: ${{ env.DOCKERHUB_USERNAME }}/athena-processor
 
       - name: Build and publish Docker - processor
         id: processor-build-and-push
@@ -75,7 +76,7 @@ jobs:
         id: monitor_meta
         uses: docker/metadata-action@v5
         with:
-          images: ${{ secrets.DOCKERHUB_USERNAME }}/athena-monitor
+          images: ${{ env.DOCKERHUB_USERNAME }}/athena-monitor
 
       - name: Build and publish Docker - monitor
         id: monitor_build-and-push