Skip to content

Commit

Permalink
remove reference to unused verification from github workflows
Browse files Browse the repository at this point in the history
A prior commit removed the separate verification image and integrated it into installer. This removes a call to that make target from the publish.yml workflow, and updates the CODEOWNERS to reference the correct ones.

Signed-off-by: Avi Deitcher <avi@deitcher.net>
  • Loading branch information
deitch committed Sep 26, 2024
1 parent f40a05d commit 8822a92
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 22 deletions.
3 changes: 1 addition & 2 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
/pkg/measure-config/ @rucoder
/pkg/memory-monitor/ @OhmSpectator
/pkg/mkimage-raw-efi/ @jsfakian @zedi-pramodh
/pkg/mkverification-raw-efi/ @jsfakian
/pkg/new-kernel/ @rucoder @rene @rouming @shjala
/pkg/newlog/ @deitch @naiming-zededa
/pkg/optee-os/ @rene
Expand Down Expand Up @@ -59,7 +58,7 @@
/pkg/storage-init/ @rouming
/pkg/u-boot/ @rene
/pkg/uefi/ @rucoder
/pkg/verification/ @jsfakian
/pkg/verifier-rs/ @jsfakian
/pkg/vtpm/ @shjala
/pkg/wwan/ @milan-zededa
/pkg/xen-tools/ @OhmSpectator @rucoder @rene @shjala
Expand Down
21 changes: 1 addition & 20 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -129,25 +129,6 @@ jobs:
dockerhub-token: ${{ secrets.RELEASE_DOCKERHUB_TOKEN }}
dockerhub-account: ${{ secrets.RELEASE_DOCKERHUB_ACCOUNT }}

verification:
if: github.event.repository.full_name == 'lf-edge/eve'
needs: packages
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
arch: [arm64, amd64]
hv: [kvm, xen]
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- uses: ./.github/actions/run-make
with:
command: "V=1 HV=${{ matrix.hv }} ZARCH=${{ matrix.arch }} LINUXKIT_PKG_TARGET=push verification"
dockerhub-token: ${{ secrets.RELEASE_DOCKERHUB_TOKEN }}
dockerhub-account: ${{ secrets.RELEASE_DOCKERHUB_ACCOUNT }}

manifest:
if: github.event.repository.full_name == 'lf-edge/eve'
runs-on: ubuntu-latest
Expand All @@ -164,7 +145,7 @@ jobs:

trigger_assets:
if: ${{ (startsWith(github.ref, 'refs/tags/')) && (github.event.repository.full_name == 'lf-edge/eve') }}
needs: [manifest, verification, eve]
needs: [manifest, eve]
uses: lf-edge/eve/.github/workflows/assets.yml@master
with:
tag_ref: ${{ github.ref_name }}

0 comments on commit 8822a92

Please sign in to comment.