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

[#1653] add CI reference job with IBM Semeru Runtimes #1654

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bmarwell
Copy link
Contributor

@bmarwell bmarwell commented Mar 23, 2023

fixes #1653

now that actions/setup-java#289 was merged

@bmarwell bmarwell force-pushed the #1653_semeru_ci_reference_job branch 2 times, most recently from bd170e2 to 0f1c445 Compare March 23, 2023 21:31
@cherylking
Copy link
Member

@bmarwell We have intermittent failures in a couple of tests (usually dev-it and generate-features-it). Does this reference build have to pass all the tests in order for the other builds to run? If so, we would probably want to exclude those two tests at a minimum to ensure the other builds run.

@bmarwell bmarwell force-pushed the #1653_semeru_ci_reference_job branch from 0f1c445 to 582c534 Compare March 23, 2023 21:33
@bmarwell
Copy link
Contributor Author

@bmarwell We have intermittent failures in a couple of tests (usually dev-it and generate-features-it). Does this reference build have to pass all the tests in order for the other builds to run? If so, we would probably want to exclude those two tests at a minimum to ensure the other builds run.

Yes, that's why I opened #1633 a while ago. That really should be fixed first. Sadly, I cannot run the tests locally due to the slightly special setup.

Also, it seems that 'semeru' is still not recognized. 😞

@bmarwell bmarwell force-pushed the #1653_semeru_ci_reference_job branch from 582c534 to f873ebd Compare March 28, 2023 11:45
@bmarwell
Copy link
Contributor Author

@cherylking the first tests fails sometimes (as expected) after 63 minutes.
Maybe we could not use all the tests all the time to save some time. WDYT?

What already works:

  • If there's a more general problem (i.e. compilation error, expected test failure) it will not occupy as much action runners as before

@cherylking
Copy link
Member

the first tests fails sometimes (as expected) after 63 minutes.
Maybe we could not use all the tests all the time to save some time. WDYT?

@bmarwell That was my concern for the reference build/test...the intermittent test failures. So I think it would be wise to exclude the dev-it and generate-features-it tests in just the reference build. That way we are more likely to get to the rest of the builds.

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.

[BUILD] add semeru build, execute matrix afterwards
2 participants