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

ci: prepare ISO artifacts in parallel with kola run #1035

Closed
wants to merge 1 commit into from
Closed

ci: prepare ISO artifacts in parallel with kola run #1035

wants to merge 1 commit into from

Conversation

bgilbert
Copy link
Contributor

No description provided.

@jlebon
Copy link
Member

jlebon commented Jul 16, 2020

Hmm, maybe with something like:

diff --git a/.cci.jenkinsfile b/.cci.jenkinsfile
index 2885b61..21e753a 100644
--- a/.cci.jenkinsfile
+++ b/.cci.jenkinsfile
@@ -1,6 +1,6 @@
 // Documentation: https://github.com/coreos/coreos-ci/blob/master/README-upstream-ci.md

-cosaPod(buildroot: true) {
+cosaPod(buildroot: true, ram: "11Gi") {
     checkout scm

     // hack to satisfy golang compiler wanting to cache things

? That'd be 1G per VM (8 for kola + 1 for blackbox + 1 for testiso) + another for overhead.

@jlebon
Copy link
Member

jlebon commented Jul 16, 2020

Actually testiso takes 4G because of coreos/fedora-coreos-tracker#407, so we'd have to make it 15Gi, which is starting to get big. One thing we could do is parallelize the cosa buildextends at least so it runs while the kola tests run, but still leave the testiso itself after the others.

@bgilbert bgilbert changed the title ci: run kola in parallel with other tests ci: prepare ISO artifacts in parallel with kola run Jul 16, 2020
@bgilbert
Copy link
Contributor Author

Turns out this makes the whole run slower. It extends the first parallel phase by ~4.5 minutes, and in the second parallel phase, buildextend + compress + testiso is faster than the blackbox tests anyway.

@bgilbert bgilbert closed this Jul 16, 2020
@bgilbert bgilbert deleted the parallel branch July 16, 2020 20:24
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.

2 participants