Attempt to fix production build (#419)

* #1341 "pin" edit tag row outside of tag list * #1341 "pin" edit tag row outside of tag list (#410) Co-authored-by: matej <matej.vavrek@m2ms.sk> * - fixed problem with tags not visually updating on observations * - implemented changes requested in #1326 * - implemented #1357 * Implements #1376, #1361, #1354 (#412) * - first implementation of #1354 * - updated message for #1354 * - cleanup for #1354 * - implemented #1361 - improvements for #1354 * - implemented #1376 - also fixed bug in download structures dialog where only first download got added to the dropdown menu * Implements purple part of #1370 (#413) * - first implementation of #1354 * - updated message for #1354 * - cleanup for #1354 * - implemented #1361 - improvements for #1354 * - implemented #1376 - also fixed bug in download structures dialog where only first download got added to the dropdown menu * - partial fix for purple release for #1370 * Partial implementation of #1357 (#415) * - first implementation of #1354 * - updated message for #1354 * - cleanup for #1354 * - implemented #1361 - improvements for #1354 * - implemented #1376 - also fixed bug in download structures dialog where only first download got added to the dropdown menu * - partial fix for purple release for #1370 * - using official backend version * - DELETE button disabled for #1357 * ci: Attempt to fix f/e actions --------- Co-authored-by: matej <matej.vavrek@m2ms.sk> Co-authored-by: boriskovar-m2ms <boris.kovar@m2ms.sk> Co-authored-by: Warren Thompson <warren.thompson+1@diamond.ac.uk> Co-authored-by: Alan Christie <alan.christie@matildapeak.com>
m2ms · Mar 13, 2024 · 6ff6173 · 6ff6173
1 parent 35eb897
commit 6ff6173
Show file tree

Hide file tree

Showing 3 changed files with 37 additions and 23 deletions.
diff --git a/.github/workflows/build-dev.yaml b/.github/workflows/build-dev.yaml
@@ -39,7 +39,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Inject slug/short variables
-        uses: rlespinasse/github-slug-action@v3.x
+        uses: rlespinasse/github-slug-action@v4
       - name: Initialise workflow variables
         id: vars
         run: |
@@ -49,8 +49,8 @@ jobs:
           echo FE_NAMESPACE=${FE_NAMESPACE}
           echo "FE_NAMESPACE=${FE_NAMESPACE}" >> $GITHUB_OUTPUT
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: Build
-        uses: docker/build-push-action@v4
+        uses: docker/build-push-action@v5
         with:
           tags: ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ env.GITHUB_REF_SLUG }}
diff --git a/.github/workflows/build-production.yaml b/.github/workflows/build-production.yaml
@@ -66,19 +66,13 @@ jobs:
       push: ${{ steps.vars.outputs.push }}
     steps:
     - name: Inject slug/short variables
-      uses: rlespinasse/github-slug-action@v3.x
+      uses: rlespinasse/github-slug-action@v4
     - name: Initialise workflow variables
       id: vars
       env:
         DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
         TRIGGER_DOWNSTREAM: ${{ secrets.TRIGGER_DOWNSTREAM }}
       run: |
-        # BE_IMAGE_TAG
-        BE_IMAGE_TAG="${{ env.BE_IMAGE_TAG }}"
-        if [ -n "${{ secrets.BE_IMAGE_TAG }}" ]; then BE_IMAGE_TAG="${{ secrets.BE_IMAGE_TAG }}"; fi
-        echo BE_IMAGE_TAG=${BE_IMAGE_TAG}
-        echo "BE_IMAGE_TAG=${BE_IMAGE_TAG}" >> $GITHUB_OUTPUT
-
         # BE_NAMESPACE
         BE_NAMESPACE="${{ env.BE_NAMESPACE }}"
         if [ -n "${{ secrets.BE_NAMESPACE }}" ]; then BE_NAMESPACE="${{ secrets.BE_NAMESPACE }}"; fi
@@ -88,7 +82,7 @@ jobs:
         # FE_IMAGE_TAG
         FE_IMAGE_TAG="${{ env.FE_IMAGE_TAG }}"
         if [ -n "${{ secrets.FE_IMAGE_TAG }}" ]; then FE_IMAGE_TAG="${{ secrets.FE_IMAGE_TAG }}"; fi
-        echo FE_IMAGE_TAG={FE_IMAGE_TAG}
+        echo FE_IMAGE_TAG=${FE_IMAGE_TAG}
         echo "FE_IMAGE_TAG=${FE_IMAGE_TAG}" >> $GITHUB_OUTPUT
 
         # FE_NAMESPACE
@@ -115,6 +109,12 @@ jobs:
         echo STACK_NAMESPACE=${STACK_NAMESPACE}
         echo "STACK_NAMESPACE=${STACK_NAMESPACE}" >> $GITHUB_OUTPUT
 
+        # What image tag are we using? 'latest' (if not tagged) or a GitHub tag?
+        TAG="latest"
+        if [[ "${{ github.ref }}" =~ ^refs/tags/ ]]; then TAG="${{ env.GITHUB_REF_SLUG }}"; fi
+        echo tag=${TAG}
+        echo "tag=${TAG}" >> $GITHUB_OUTPUT
+
         # Do we push, i.e. is DOCKERHUB_USERNAME defined?
         echo push=${{ env.DOCKERHUB_USERNAME != '' }}
         echo "push=${{ env.DOCKERHUB_USERNAME != '' }}" >> $GITHUB_OUTPUT
@@ -124,12 +124,19 @@ jobs:
         echo "trigger=${{ env.TRIGGER_DOWNSTREAM == 'yes' }}" >> $GITHUB_OUTPUT
 
     - name: Checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
+    - name: Lint Dockerfile
+      uses: hadolint/hadolint-action@v3.1.0
+      with:
+        dockerfile: Dockerfile
     - name: Build
-      uses: docker/build-push-action@v4
+      uses: docker/build-push-action@v5
       with:
-        tags: ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.FE_IMAGE_TAG }}
-    - name: Login to Docker Hub
+        context: .
+        tags: |
+          ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.tag }}
+          ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:stable
+    - name: Login to DockerHub
       if: steps.vars.outputs.push == 'true'
       uses: docker/login-action@v3
       with:
@@ -138,7 +145,8 @@ jobs:
     - name: Push
       if: steps.vars.outputs.push == 'true'
       run: |
-        docker push ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.FE_IMAGE_TAG }}
+        docker push ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.tag }}
+        docker push ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:stable
 
     # Trigger the stack for this build,
     # identifying the 'stable' backend (or whatever BE_IMAGE_TAG was used)

diff --git a/.github/workflows/build-staging.yaml b/.github/workflows/build-staging.yaml
@@ -71,7 +71,7 @@ jobs:
       push: ${{ steps.vars.outputs.push }}
     steps:
       - name: Inject slug/short variables
-        uses: rlespinasse/github-slug-action@v3.x
+        uses: rlespinasse/github-slug-action@v4
       - name: Initialise workflow variables
         id: vars
         env:
@@ -120,6 +120,12 @@ jobs:
           echo STACK_NAMESPACE=${STACK_NAMESPACE}
           echo "STACK_NAMESPACE=${STACK_NAMESPACE}" >> $GITHUB_OUTPUT
 
+          # What image tag are we using? 'latest' (if not tagged) or a GitHub tag?
+          TAG="latest"
+          if [[ "${{ github.ref }}" =~ ^refs/tags/ ]]; then TAG="${{ env.GITHUB_REF_SLUG }}"; fi
+          echo tag=${TAG}
+          echo "tag=${TAG}" >> $GITHUB_OUTPUT
+
           # Do we push, i.e. is DOCKERHUB_USERNAME defined?
           echo push=${{ env.DOCKERHUB_USERNAME != '' }}
           echo "push=${{ env.DOCKERHUB_USERNAME != '' }}" >> $GITHUB_OUTPUT
@@ -129,11 +135,12 @@ jobs:
           echo "trigger=${{ env.TRIGGER_DOWNSTREAM == 'yes' }}" >> $GITHUB_OUTPUT
 
       - name: Checkout
-        uses: actions/checkout@v3
-      - name: Build
-        uses: docker/build-push-action@v4
+        uses: actions/checkout@v4
+      - name: Docker build
+        uses: docker/build-push-action@v5
         with:
-          tags: ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.FE_IMAGE_TAG }}
+          context: .
+          tags: ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.tag }}
       - name: Login to Docker Hub
         if: steps.vars.outputs.push == 'true'
         uses: docker/login-action@v3
@@ -142,8 +149,7 @@ jobs:
           password: ${{ secrets.DOCKERHUB_TOKEN }}
       - name: Push
         if: steps.vars.outputs.push == 'true'
-        run: |
-          docker push ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.FE_IMAGE_TAG }}
+        run: docker push ${{ steps.vars.outputs.FE_NAMESPACE }}/fragalysis-frontend:${{ steps.vars.outputs.tag }}
 
       # Trigger the stack for this build,
       # identifying the 'stable' backend (or whatever BE_IMAGE_TAG has been set)