Chore(ci): Proper tag for docker img (#182)

* proper tag

* add build on commit to main and add logic for tag latest & numeric tag

Author: hoangdv2429

.github/workflows/build_and_push.yml

@@ -2,17 +2,19 @@ name: Build and Push Multi-Arch Images
 
 on:
   push:
+    branches:
+      - main
     tags:
       - 'v[0-9]+.[0-9]+.[0-9]+'
 
 env:
   REGISTRY_IMAGE: ghcr.io/lay3rlabs/layer-sdk
-  IMAGE_TAG: ${{ github.sha }}
 
 jobs:
   build_amd64:
     name: Build and Push AMD64 Image
     runs-on: linux-8-core
+    if: ${{ github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/v') }}
     permissions:
       contents: read
       packages: write
@@ -21,6 +23,20 @@ jobs:
       - name: Checkout the repo
         uses: actions/checkout@v4
 
+      - name: Set IMAGE_TAG
+        shell: bash
+        run: |
+          if [[ "${GITHUB_REF_TYPE}" == "tag" ]]; then
+            TAG="${GITHUB_REF_NAME#v}"
+            IMAGE_TAG="${TAG//[^a-zA-Z0-9_.-]/-}"
+          elif [[ "${GITHUB_REF_NAME}" == "main" ]]; then
+            IMAGE_TAG="latest"
+          else
+            echo "Not on main branch or tag, skipping build."
+            exit 1
+          fi
+          echo "IMAGE_TAG=${IMAGE_TAG}" >> $GITHUB_ENV
+
       - name: Log in to the Container registry
         uses: docker/login-action@v3
         with:
@@ -33,14 +49,15 @@ jobs:
         with:
           context: .
           file: ./docker/Dockerfile.slay3rd
-          push: ${{ github.event_name != 'pull_request' }}
+          push: true
           tags: ${{ env.REGISTRY_IMAGE }}:${{ env.IMAGE_TAG }}-amd64
           platforms: linux/amd64
           provenance: false
 
   build_arm64:
     name: Build and Push ARM64 Image
     runs-on: arm64-builder
+    if: ${{ github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/v') }}
     permissions:
       contents: read
       packages: write
@@ -49,6 +66,20 @@ jobs:
       - name: Checkout the repo
         uses: actions/checkout@v4
 
+      - name: Set IMAGE_TAG
+        shell: bash
+        run: |
+          if [[ "${GITHUB_REF_TYPE}" == "tag" ]]; then
+            TAG="${GITHUB_REF_NAME#v}"
+            IMAGE_TAG="${TAG//[^a-zA-Z0-9_.-]/-}"
+          elif [[ "${GITHUB_REF_NAME}" == "main" ]]; then
+            IMAGE_TAG="latest"
+          else
+            echo "Not on main branch or tag, skipping build."
+            exit 1
+          fi
+          echo "IMAGE_TAG=${IMAGE_TAG}" >> $GITHUB_ENV
+
       - name: Log in to the Container registry
         uses: docker/login-action@v3
         with:
@@ -61,7 +92,7 @@ jobs:
         with:
           context: .
           file: ./docker/Dockerfile.slay3rd
-          push: ${{ github.event_name != 'pull_request' }}
+          push: true
           tags: ${{ env.REGISTRY_IMAGE }}:${{ env.IMAGE_TAG }}-arm64
           platforms: linux/arm64
           provenance: false
@@ -70,7 +101,7 @@ jobs:
     name: Create and Push Multi-Arch Manifest
     runs-on: ubuntu-latest
     needs: [build_amd64, build_arm64]
-    if: ${{ github.event_name != 'pull_request' }}
+    if: ${{ github.ref == 'refs/heads/main' || startsWith(github.ref, 'refs/tags/v') }}
     permissions:
       contents: read
       packages: write
@@ -79,14 +110,19 @@ jobs:
       - name: Checkout the repo
         uses: actions/checkout@v4
 
-      - name: Docker metadata
-        id: meta
-        uses: docker/metadata-action@v5
-        with:
-          images: ${{ env.REGISTRY_IMAGE }}
-          tags: |
-            type=ref,event=branch
-            type=sha
+      - name: Set IMAGE_TAG
+        shell: bash
+        run: |
+          if [[ "${GITHUB_REF_TYPE}" == "tag" ]]; then
+            TAG="${GITHUB_REF_NAME#v}"
+            IMAGE_TAG="${TAG//[^a-zA-Z0-9_.-]/-}"
+          elif [[ "${GITHUB_REF_NAME}" == "main" ]]; then
+            IMAGE_TAG="latest"
+          else
+            echo "Not on main branch or tag, skipping manifest creation."
+            exit 1
+          fi
+          echo "IMAGE_TAG=${IMAGE_TAG}" >> $GITHUB_ENV
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3
@@ -99,35 +135,8 @@ jobs:
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Create and push multi-arch manifest
-        env:
-          REGISTRY_IMAGE: ${{ env.REGISTRY_IMAGE }}
-          IMAGE_TAG: ${{ env.IMAGE_TAG }}
-          TAGS: ${{ steps.meta.outputs.tags }}
         run: |
-          echo "REGISTRY_IMAGE: $REGISTRY_IMAGE"
-          echo "IMAGE_TAG: $IMAGE_TAG"
-          echo "TAGS: $TAGS"
-        
-            # Initialize TAG_ARGS with the primary tag
-            TAG_ARGS="-t ${REGISTRY_IMAGE}:${IMAGE_TAG}"
-        
-            # Check if TAGS is not empty
-            if [ -n "$TAGS" ]; then
-              # Split TAGS into an array
-              IFS=',' read -ra TAG_ARRAY <<< "$TAGS"
-              for FULL_TAG in "${TAG_ARRAY[@]}"; do
-                # Extract the tag name from the full image reference
-                TAG=${FULL_TAG##*:}
-                # Append to TAG_ARGS if TAG is not empty
-                if [ -n "$TAG" ]; then
-                  TAG_ARGS="$TAG_ARGS -t ${REGISTRY_IMAGE}:$TAG"
-                fi
-              done
-            fi
-        
-            echo "TAG_ARGS: $TAG_ARGS"
-        
-            # Create and push the multi-arch manifest with all tags
-            docker buildx imagetools create $TAG_ARGS \
-              ${REGISTRY_IMAGE}:${IMAGE_TAG}-amd64 \
-              ${REGISTRY_IMAGE}:${IMAGE_TAG}-arm64
+          docker buildx imagetools create \
+            --tag ${{ env.REGISTRY_IMAGE }}:${{ env.IMAGE_TAG }} \
+            ${{ env.REGISTRY_IMAGE }}:${{ env.IMAGE_TAG }}-amd64 \
+            ${{ env.REGISTRY_IMAGE }}:${{ env.IMAGE_TAG }}-arm64