From 0aa26026bd428c25528447c97193fb8e22cbc183 Mon Sep 17 00:00:00 2001
From: Valentin Sawadski <valentin@sawadski.eu>
Date: Wed, 15 Sep 2021 18:26:17 +0100
Subject: [PATCH] Feat: Build darknet release based on git tag and ignore
 pushes to master branch

This allows us to use specific builds in ODC instead of using whatever was
in master at the time of building the ODC docker image

Closes https://github.com/opendatacam/opendatacam/issues/436
---
 .github/workflows/ci-cd.yml | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci-cd.yml b/.github/workflows/ci-cd.yml
index 30ff8199af1..e8d99b5808e 100644
--- a/.github/workflows/ci-cd.yml
+++ b/.github/workflows/ci-cd.yml
@@ -9,6 +9,8 @@ on:
   pull_request:
     branches-ignore:
       - 'gh-pages'
+  release:
+    types: [created]
 
 jobs:
   # Build docker images
@@ -16,12 +18,14 @@ jobs:
     runs-on: ubuntu-latest
     # needs: build-and-test-code
     env:
-      # Publish to Docker Hub if we have a change in the development or master branch
-      DOCKERHUB_PUBLISH: ${{ github.event_name == 'push' && (github.event.ref == 'refs/heads/development' || github.event.ref == 'refs/heads/master') && secrets.DOCKERHUB_USERNAME != '' }}
+      # Only publish to Docker Hub if the dockerhub username is set, and either happens
+      #
+      # - we have a change in the development branch
+      # - we have a new tag
+      DOCKERHUB_PUBLISH: ${{ (github.event_name == 'push' && (github.event.ref == 'refs/heads/development' || startsWith(github.event.ref, 'refs/tags/')) || github.event_name == 'release') && secrets.DOCKERHUB_USERNAME != '' }}
       # In case we run a pull request, the secrets are not available to us. Therefore check first
       # and assign a 'dummy' dockerhub username
       DOCKERHUB_USERNAME: ${{ ( secrets.DOCKERHUB_USERNAME != '' && secrets.DOCKERHUB_USERNAME ) || 'dummy' }}
-      DOCKERHUB_TAG_SUFFIX: ${{ ( github.event.ref == 'refs/heads/development' && format('{0}-{1}', matrix.platform, 'development') ) || ( github.event.ref == 'refs/heads/master' && matrix.platform ) || format('{0}-{1}', matrix.platform, 'dummy') }}
     strategy:
       matrix:
         include:
@@ -46,6 +50,17 @@ jobs:
           # Use the secrets directly and not the validated environment DOCKERHUB_USERNAME
           username: ${{ secrets.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
+      - name: Determine dockerhub tag
+        id: dockerhub-tag
+        # Determine the dockerhub tag based on if our current working tree points to a tag or not
+        # the tags will either be `<tag>-<platform>` or `development-<platform>`
+        run: |
+          GIT_TAG=$(git tag -l --points-at $(git log -n 1 --format=%H))
+          if [ ! -z $GIT_TAG ]; then
+            echo "::set-output name=dockerhub-tag::$GIT_TAG-${{ matrix.platform }}"
+          else
+            echo "::set-output name=dockerhub-tag::development-${{ matrix.platform }}"
+          fi
       - name: Build and push
         uses: docker/build-push-action@v2
         with:
@@ -53,4 +68,4 @@ jobs:
           file: docker/build/${{ matrix.platform }}/Dockerfile
           platforms: ${{ matrix.docker-platform }}
           push: ${{ env.DOCKERHUB_PUBLISH == 'true' }}
-          tags: ${{ env.DOCKERHUB_USERNAME }}/opendatacam-darknet-base:${{ env.DOCKERHUB_TAG_SUFFIX }}
+          tags: ${{ env.DOCKERHUB_USERNAME }}/opendatacam-darknet-base:${{ steps.dockerhub-tag.outputs.dockerhub-tag }}