[CI] Add use-snapshot option to reuse pre-built Quarkus snapshots#960
[CI] Add use-snapshot option to reuse pre-built Quarkus snapshots#960zakkak wants to merge 4 commits into
Conversation
cb854c8 to
51358aa
Compare
|
As suspected it fails due to some missing dependencies This is because these artifacts are not deployed as they are only used by quarkus core integration tests. I need to adapt the workflow to build those explicitly for snapshots. The workflow also spends ~1m downloading deps because when we use a Quarkus snaphost we no longer use the GH cache. Might be worth fixing this as well. |
|
Things look better with Some tests still failing, e.g. gradle tooling fails with |
1cda064 to
17e1bfb
Compare
|
Hmm, we usually build Quarkus with a new version of GraalVM SDK (in order to ensure they still support the right JDK minimal version). With this move we no longer get this benefit or do we? |
True, with this move we no longer test whether Quarkus can build with the latest GraalVM/Mandrel SDK. We currently do this when |
Sounds like a good plan to me. Every two weeks should be enough as a ball-park frequency of asserting nothing broke in that regard. |
25192eb to
324fcbc
Compare
Adaptation of https://github.com/quarkusio/install-quarkus-snapshots-action/ to reduce CI overhead. Assisted-by: Bob
Also enables assertions in this jobs to avoid the need for a monthly workflow as well.
324fcbc to
ff091f5
Compare
Adaptation of https://github.com/quarkusio/install-quarkus-snapshots-action/ to reduce CI overhead.
Assisted-by: Bob
Being tested in https://github.com/zakkak/mandrel/actions/runs/25021568549