Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use self hosted wolfi runners for pre/postsubmit builds #52

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

joshrwolf
Copy link
Member

@joshrwolf joshrwolf commented Aug 29, 2023

refactors how we build the individual images to better align with our new custom multi arch runners.

previous:

  • matrix fan out/in with qemu builds

new:

  • reusable github workflows to create independent workflows for each independent image
  • each image workflow fans out to the appropriate arch melange build, then fans in for the apko publish step

it's a significant refactor, but I think this is easier to follow what's going on

@joshrwolf joshrwolf requested a review from a team as a code owner August 29, 2023 20:18
@joshrwolf joshrwolf requested review from tcnghia and imjasonh and removed request for a team August 29, 2023 20:18
@joshrwolf joshrwolf force-pushed the wolfi-runners branch 26 times, most recently from 88c5950 to d15b4b5 Compare August 30, 2023 15:23
@joshrwolf joshrwolf force-pushed the wolfi-runners branch 5 times, most recently from 338ccea to 1886a31 Compare August 30, 2023 16:00
@joshrwolf joshrwolf force-pushed the wolfi-runners branch 4 times, most recently from 9aea899 to ce737ff Compare August 30, 2023 16:39
Comment on lines -8 to -13
inputs:
only:
description: 'Specific image name to build'
type: string
required: false
default: ''
Copy link
Member Author

@joshrwolf joshrwolf Aug 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removing this is part laziness of figuring out the magical if: github actions incantation, but also part because these builds are much faster and far more independent now, so it's simpler to just rebuild everything on a trigger

Comment on lines +11 to +14
permissions:
id-token: write
packages: write
contents: read
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't actually test that this is WAI until this lands, so I'll monitor this when it lands

@kaniini kaniini merged commit 6cf3573 into wolfi-dev:main Sep 1, 2023
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants