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

Add nightly builds workflow and allow RC releases #151

Merged
merged 3 commits into from
Jun 10, 2021
Merged

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Jun 10, 2021

No description provided.

Signed-off-by: Hidde Beydals <hello@hidde.co>
Signed-off-by: Hidde Beydals <hello@hidde.co>
@hiddeco hiddeco added the area/ci CI related issues and pull requests label Jun 10, 2021
@stefanprodan
Copy link
Member

Let's disable the multi-arch build for each PR.

Signed-off-by: Hidde Beydals <hello@hidde.co>
@squaremo
Copy link
Member

It'd be good to see a rationale for this, especially for disabling multi-arch builds.

@stefanprodan
Copy link
Member

We've moved the multi-arch build to nightly, as the build takes anywhere from 20 to 35 minutes making the release process too slow. Note that this is the only repo where I've enabled multi-arch builds for PRs. This PR brings IRC CI on pair with all the others controllers.

@hiddeco
Copy link
Member Author

hiddeco commented Jun 10, 2021

In the other Flux projects we were only building the multi-arch image during the release, this has (unsurprisingly) resulted in two releases that failed at the last moment.

To mitigate against this, we introduced a "nightly" build that runs once every night and on-demand, so that small PRs and/or things that have little relevance to 32-bit do not burn useless minutes in CI, as the build time is increased by ^5-10 for multi-arch builds.

When I created this PR I was unaware the multi-arch build was already run for the "normal build", I was just trying to get things in-line with our other projects.

@hiddeco
Copy link
Member Author

hiddeco commented Jun 10, 2021

The other option I suggested instead of "nightly" is to run the multi-arch build on merges to main, but not for pull requests themselves. That may be more up to your likening.

@stefanprodan
Copy link
Member

When I created this PR I was unaware the multi-arch build was already run for the "normal build", I was just trying to get things in-line with our other projects.

Yea it's fault, I should've undo it here after the Badger bug fixes.

@squaremo
Copy link
Member

OK, thanks for explaining @hiddeco and @stefanprodan. I do prefer the "full build on merge to main" alternative, but nightlies is fine too.

@hiddeco
Copy link
Member Author

hiddeco commented Jun 10, 2021

What if we refactor things in a next iteration so that merges on main always build a multi-arch image, while the "nightly" also pushes the image (as we did for Flux in the past so that people have access to "early releases")?

@hiddeco
Copy link
Member Author

hiddeco commented Jun 10, 2021

Merging this to align things, fluxcd/flux2#1498 discusses a future revision.

@hiddeco hiddeco merged commit 9fe157c into main Jun 10, 2021
@hiddeco hiddeco deleted the nightly-builds branch June 10, 2021 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants