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

Reduce CI load on failing PR #4952

Merged
merged 1 commit into from
Oct 29, 2019

Conversation

stuartwdouglas
Copy link
Member

This PR changes the staging of CI jobs to not run native tests
if the JVM tests fail. It also removes the dedicated 'build for
native' job, and instead uses the results of the JDK8 build
to run the native build, further reducing the load.

@stuartwdouglas stuartwdouglas force-pushed the ci-staging branch 2 times, most recently from b18c54b to f7572ea Compare October 28, 2019 23:13
This changes the CI pipeline to run JDK8 first,
and only run the rest of the jobs if it passes.

This will greatly reduce the load for failing PR's,
at the expense of increasing the total time when CI
is not busy and the PR is passing.

It also removes the native image build, and just uses
the output of the standard JDK8 Linux build.
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Makes sense.

@gsmet gsmet added this to the 0.27.0 milestone Oct 29, 2019
@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Oct 29, 2019
@stuartwdouglas stuartwdouglas merged commit 0ac87f1 into quarkusio:master Oct 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants