Skip to content

Commit

Permalink
Merge pull request #4285 from ty-dc/fix/trivy-db
Browse files Browse the repository at this point in the history
Fix TOOMANYREQUESTS failure in Trivy Action
  • Loading branch information
weizhoublue authored Nov 25, 2024
2 parents 747baa6 + b911148 commit 4acc850
Show file tree
Hide file tree
Showing 4 changed files with 27 additions and 25 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build-image-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,7 @@ jobs:
# docker cache after the workflow "Image CI Cache Cleaner" was terminated.
push: ${{ env.push }}
platforms: linux/amd64
outputs: type=tar,dest=/tmp/${{ matrix.name }}-race.tar
outputs: type=docker,dest=/tmp/${{ matrix.name }}-race.tar
github-token: ${{ secrets.WELAN_PAT }}
tags: |
${{ env.ONLINE_REGISTER }}/${{ github.repository }}/${{ matrix.name }}-ci:${{ env.tag }}-race
Expand Down Expand Up @@ -231,7 +231,7 @@ jobs:
push: ${{ env.push }}
platforms: linux/amd64
github-token: ${{ secrets.WELAN_PAT }}
outputs: type=tar,dest=/tmp/${{ matrix.name }}-race.tar
outputs: type=docker,dest=/tmp/${{ matrix.name }}-race.tar
tags: |
${{ env.ONLINE_REGISTER }}/${{ github.repository }}/${{ matrix.name }}-ci:${{ env.tag }}-race
build-args: |
Expand Down
9 changes: 7 additions & 2 deletions .github/workflows/e2e-init.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -122,8 +122,13 @@ jobs:
for ITEM in $TAR_FILES ; do
IMAGE_NAME=${ITEM%*.tar}
echo ${IMAGE_NAME}
cat test/.download/${ITEM} | docker import - ${IMAGE_NAME}:${{ inputs.image_tag }}
docker load -i test/.download/${ITEM}
echo "list docker images" && docker images
ITEM_IMAGE_ID=$(docker images | grep ${IMAGE_NAME%*-race}| grep ${{ inputs.image_tag }} | awk '{print $3}')
docker tag ${ITEM_IMAGE_ID} ${IMAGE_NAME}:${{ inputs.image_tag }}
done
echo "list all docker images"
docker images
# test against commit version
# https://github.com/kubernetes-sigs/kind/issues/2863
Expand Down Expand Up @@ -164,7 +169,7 @@ jobs:
else
echo "RUN_SETUP_KIND_CLUSTER_PASS=false" >> $GITHUB_ENV
fi
if [ -f "test/e2edebugLog" ] ; then
if [ -f "test/e2edebugLog.txt" ] ; then
echo "UPLOAD_SETUP_KIND_CLUSTER_LOG=true" >> $GITHUB_ENV
else
echo "UPLOAD_SETUP_KIND_CLUSTER_LOG=false" >> $GITHUB_ENV
Expand Down
38 changes: 18 additions & 20 deletions .github/workflows/trivy-scan-image.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,24 +35,22 @@ jobs:
name: image-tar-spiderpool-controller
path: test/.download

- name: Load And Scan Images
run: |
TAR_FILES=` ls test/.download `
echo $TAR_FILES
for ITEM in $TAR_FILES ; do
IMAGE_NAME=${ITEM%*.tar}
echo ${IMAGE_NAME}
cat test/.download/${ITEM} | docker import - ${IMAGE_NAME}:${{ inputs.image_tag }}
echo "---------trivy checkout image ${IMAGE_NAME}:${{ inputs.image_tag }} --------------------"
make lint_image_trivy -e IMAGE_NAME=${IMAGE_NAME}:${{ inputs.image_tag }} \
|| { echo "RUN_IMAGE_TRIVY_FAIL=true" >> $GITHUB_ENV ; echo "error, image ${IMAGE_NAME}:${{ inputs.image_tag }} is bad" ; }
done
- name: List downloaded files
run: ls -al test/.download

- name: Show Trivy Scan Report
run: |
if [ "${{ env.RUN_IMAGE_TRIVY_FAIL }}" == "true" ] ; then
echo "error, image is not secure, see detail on Step 'Load And Scan Images' "
exit 1
else
exit 0
fi
# https://github.com/aquasecurity/trivy-action/issues/389
- name: load and scan spiderpool-agent image
uses: aquasecurity/[email protected]
env:
TRIVY_DB_REPOSITORY: public.ecr.aws/aquasecurity/trivy-db,ghcr.io/aquasecurity/trivy-db
with:
input: test/.download/spiderpool-agent-race.tar
severity: 'CRITICAL,HIGH'

- name: load and scan spiderpool-controller image
uses: aquasecurity/[email protected]
env:
TRIVY_DB_REPOSITORY: public.ecr.aws/aquasecurity/trivy-db,ghcr.io/aquasecurity/trivy-db
with:
input: test/.download/spiderpool-controller-race.tar
severity: 'CRITICAL,HIGH'
1 change: 0 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -501,4 +501,3 @@ lint_chart_trivy:
.PHONY: build-chart
build-chart:
@ cd charts ; make

0 comments on commit 4acc850

Please sign in to comment.