Skip to content

Add switch for running Sonobuoy tests in parallel#1117

Merged
mbuechse merged 2 commits intomainfrom
feat/sonobuoy-parallel
Mar 19, 2026
Merged

Add switch for running Sonobuoy tests in parallel#1117
mbuechse merged 2 commits intomainfrom
feat/sonobuoy-parallel

Conversation

@mbuechse
Copy link
Contributor

@mbuechse mbuechse commented Feb 24, 2026

Many people at the SCS hackathon observed that Sonobuoy took a lot of time. Having it run its tests in parallel might help that.

@mbuechse mbuechse force-pushed the feat/sonobuoy-parallel branch 2 times, most recently from 847d773 to de7a67e Compare March 4, 2026 18:52
@depressiveRobot depressiveRobot assigned mbuechse and unassigned mbuechse Mar 9, 2026
garloff added a commit to SovereignCloudStack/scs-training-kaas-scripts that referenced this pull request Mar 12, 2026
@garloff
Copy link
Member

garloff commented Mar 13, 2026

OK, on my CiaB (8-core) test with a 3+3 cluster, the --e2eparallel=true flag caused the runtime to go from 2hrs to 38mins. No failures.

Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
as decided by SIG Standardization/Certification on 2026-03-12

Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
@mbuechse mbuechse force-pushed the feat/sonobuoy-parallel branch from de7a67e to a7056f1 Compare March 14, 2026 14:55
@mbuechse mbuechse requested a review from gerbsen March 14, 2026 15:03
@fkr fkr self-assigned this Mar 18, 2026
@fkr
Copy link
Member

fkr commented Mar 18, 2026

@gerbsen see my comment - with that in mind, can you remove the 'change requested'?

Copy link
Contributor

@gerbsen gerbsen left a comment

Choose a reason for hiding this comment

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

IMHO this could be an ideal test ballon for auto-generated tests. Would have to be checked manually as well of course. I tried this with Claude an attach the result for future reference. I didn't test these tests myself.

scs-compliance-check-test.yaml
scs_compliance_check_test.py
report.yaml

@mbuechse
Copy link
Contributor Author

mbuechse commented Mar 19, 2026

I think is goes without saying that feeding our code to Claude is not something I appreciate. The test code it produced is 885 lines of boilerplate code that would need maintenance, for instance, it checks whether all command-line options are documented in the usage output, but the list of options is hard-coded, so it's no use at all. I can't stress enough how I oppose this.

@mbuechse mbuechse merged commit 8be38e6 into main Mar 19, 2026
10 checks passed
@mbuechse mbuechse deleted the feat/sonobuoy-parallel branch March 19, 2026 14:11
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.

4 participants