-
-
Notifications
You must be signed in to change notification settings - Fork 71
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
Enable SBOM strace for all Linux platforms #1065
Enable SBOM strace for all Linux platforms #1065
Conversation
Signed-off-by: Andrew Leonard <[email protected]>
Thank you for creating a pull request!Please check out the information below if you have not made a pull request here before (or if you need a reminder how things work). Code Quality and Contributing GuidelinesIf you have not done so already, please familiarise yourself with our Contributing Guidelines and Code Of Conduct, even if you have contributed before. TestsGithub actions will run a set of jobs against your PR that will lint and unit test your changes. Keep an eye out for the results from these on the latest commit you submitted. For more information, please see our testing documentation. In order to run the advanced pipeline tests (executing a set of mock pipelines), it requires an admin to post |
Signed-off-by: Andrew Leonard <[email protected]>
run tests |
jdk8u mac fails for un-related issue:
|
Signed-off-by: Andrew Leonard <[email protected]>
run tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm ok for this to go in for now, but noting that this will have a significant impact on the build times by reducing the jobs, and also this is hard coding to values which won't be great if we run on a machine with smaller numbers of cores.
@andrew-m-leonard holding off merging since the jdk 11 on Linux went over 8 hours, was this new feature the cause? not sure :-) |
PR TESTER RESULT ❎ Some pipelines failed or the job was aborted! ❎ |
thanks @karianna yes, so that's interesting, i've just anaylsed why jdk11u x64Linux took so long, and I think it is because it is using ccache, which results in thousands of these effectively unknown package C source being used:
Need to think about that one |
Signed-off-by: Andrew Leonard <[email protected]>
run tests |
PR TESTER RESULT ❎ Some pipelines failed or the job was aborted! ❎ |
Signed-off-by: Andrew Leonard <[email protected]>
run tests |
PR TESTER RESULT ✅ All pipelines passed! ✅ |
run tests |
PR TESTER RESULT ❎ Some pipelines failed or the job was aborted! ❎ |
8107467
to
b019b42
Compare
run tests |
PR TESTER RESULT ✅ All pipelines passed! ✅ |
Fixes #1064
Enable SBOM strace for all Linux platforms (except riscv64 which fails in emulation environment)
Note:
Also, fix "cleanWorkspaceBuildOutputAfterBuild" so it cleans straceOutput, since it is about 3Gb in size!