ci: drop linux/arm/v7 from the platforms for building container-images

There is not need for `secrets.BUILD_PLATFORMS`, as the platforms are not confidential. Instead a variable has been created in the GitHub repository settings for it now. The `secrets.BUILD_PLATFORMS` includes linux/arm/v7, which is a slow and phased out architecture. All reasonable ARM platforms are linux/arm64 now. Dropping the linux/arm/v7 platform makes building the container images much faster. Closes: #719 Signed-off-by: Niels de Vos <[email protected]>
csi-addons · Nov 28, 2024 · 7e9ae42 · 7e9ae42
1 parent f037599
commit 7e9ae42
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/.github/workflows/build-push.yaml b/.github/workflows/build-push.yaml
@@ -65,7 +65,7 @@ jobs:
         with:
           context: .
           file: Dockerfile
-          platforms: ${{ secrets.BUILD_PLATFORMS }}
+          platforms: ${{ vars.BUILD_PLATFORMS }}
           push: true
           tags: quay.io/csiaddons/k8s-controller:latest
 
@@ -95,6 +95,6 @@ jobs:
         with:
           context: .
           file: build/Containerfile.sidecar
-          platforms: ${{ secrets.BUILD_PLATFORMS }}
+          platforms: ${{ vars.BUILD_PLATFORMS }}
           push: true
           tags: quay.io/csiaddons/k8s-sidecar:latest
diff --git a/.github/workflows/test-build.yaml b/.github/workflows/test-build.yaml
@@ -52,7 +52,7 @@ jobs:
         with:
           context: .
           file: Dockerfile
-          platforms: ${{ secrets.BUILD_PLATFORMS }}
+          platforms: ${{ vars.BUILD_PLATFORMS }}
           push: false
           tags: quay.io/csiaddons/k8s-controller:latest
 
@@ -74,6 +74,6 @@ jobs:
         with:
           context: .
           file: build/Containerfile.sidecar
-          platforms: ${{ secrets.BUILD_PLATFORMS }}
+          platforms: ${{ vars.BUILD_PLATFORMS }}
           push: false
           tags: quay.io/csiaddons/k8s-sidecar:latest
diff --git a/docs/ci.md b/docs/ci.md
@@ -12,10 +12,10 @@ supports.
 For this configuration, a new Secret should be created in the GitHub
 Settings of the repository. 'Normal' environment variables seem not possible.
 
-An example of the GitHub Secret that will build the container-images on AMD64,
-and both 32-bit and 64-bit Arm platforms:
+An example of the GitHub Variable that will build the container-images on
+AMD64, and both 32-bit and 64-bit Arm platforms:
 
-- `BUILD_PLATFORMS`: `linux/amd64,linux/arm64,linux/arm/v7`
+- `BUILD_PLATFORMS`: `linux/amd64,linux/arm64`
 
 Detailed steps on creating the GitHub Secret can be found in [the GitHub
 Documentation][gh_doc_secret].