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

Move cache extensions from root modules to spread load evenly in matrix (especially as way to mitigate native build OOM in dailys) #1533

Merged

Conversation

michalvavrik
Copy link
Member

Summary

I don't know what to do about dailys - https://github.com/quarkus-qe/quarkus-test-suite/actions/workflows/daily.yaml it seems to me like either we need to introduce some cleaning of resources, or we need to spread load to other runners. I already had 2 experimental attempts #1529 and #1523 without success.

But what is worse, new addition is coming right into root modules where we fight resources: #1524

I propose to separate cache modules as that way we can easily couple them with other profiles in matrix. Native executable build time is very volatile, but building HTTP modules still takes less than root-modules. Sometimes significantly less, see https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/6819612674/job/18547594454 2 hours less.

I'll follow-up with Jenkins updates if this goes in.

Please select the relevant options.

  • Bug fix (non-breaking change which fixes an issue)
  • Dependency update
  • Refactoring
  • Backport
  • New scenario (non-breaking change which adds functionality)
  • This change requires a documentation update
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@michalvavrik michalvavrik requested a review from jedla97 November 20, 2023 16:28
Copy link
Member

@jedla97 jedla97 left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

@michalvavrik michalvavrik merged commit 8bd2f13 into quarkus-qe:main Nov 20, 2023
8 checks passed
@michalvavrik michalvavrik deleted the feature/introduce-cache-modules branch November 20, 2023 18:25
@michalvavrik michalvavrik added triage/backport-3.2 RHBQ Ghost LTS release and removed triage/backport-3.2 RHBQ Ghost LTS release labels Nov 21, 2023
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