From 445b389b16f3957ba3efb6cfe720d7d9de454c2e Mon Sep 17 00:00:00 2001 From: Zvonko Kaiser Date: Wed, 19 Jun 2024 07:46:51 +0000 Subject: [PATCH 1/2] ci: remove sudo from arm64 build We can now do the same for arm64 that we did for amd64 and remove the sudo cp. Signed-off-by: Zvonko Kaiser --- .github/workflows/build-kata-static-tarball-arm64.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build-kata-static-tarball-arm64.yaml b/.github/workflows/build-kata-static-tarball-arm64.yaml index c280da9ff1..57fa4a1b12 100644 --- a/.github/workflows/build-kata-static-tarball-arm64.yaml +++ b/.github/workflows/build-kata-static-tarball-arm64.yaml @@ -68,7 +68,7 @@ jobs: make "${KATA_ASSET}-tarball" build_dir=$(readlink -f build) # store-artifact does not work with symlink - sudo cp -r "${build_dir}" "kata-build" + mkdir -p kata-build && cp "${build_dir}"/kata-static-${KATA_ASSET}*.tar.* kata-build/. env: KATA_ASSET: ${{ matrix.asset }} TAR_OUTPUT: ${{ matrix.asset }}.tar.gz From 3beb460a9772df16a8f7a27c606b0207a3336574 Mon Sep 17 00:00:00 2001 From: Zvonko Kaiser Date: Wed, 19 Jun 2024 07:48:00 +0000 Subject: [PATCH 2/2] ci: Remove not needed chown for arm64 Now that all artifacts are owned by $USER no extra step needed to adjust ownership Signed-off-by: Zvonko Kaiser --- .github/workflows/build-kata-static-tarball-arm64.yaml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/.github/workflows/build-kata-static-tarball-arm64.yaml b/.github/workflows/build-kata-static-tarball-arm64.yaml index 57fa4a1b12..bbe4017a61 100644 --- a/.github/workflows/build-kata-static-tarball-arm64.yaml +++ b/.github/workflows/build-kata-static-tarball-arm64.yaml @@ -40,10 +40,6 @@ jobs: - shim-v2 - virtiofsd steps: - - name: Adjust a permission for repo - run: | - sudo chown -R $USER:$USER $GITHUB_WORKSPACE - - name: Login to Kata Containers quay.io if: ${{ inputs.push-to-registry == 'yes' }} uses: docker/login-action@v3