-
Notifications
You must be signed in to change notification settings - Fork 27
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
[1.3.z] Bump Quarkus and Quarkus config version to 3.2.12.Final, remove helm and disable CLI test #1095
Conversation
run tests |
Windows was failing because of helm which was removed in main, backporting this commit |
71efd4f
to
0865ecd
Compare
a281d13
to
e1960b1
Compare
CLI error are caused by quarkusio/quarkus#37808 (wasn't been backported to 3.2) this should not affect newer Quarkus version. Try it localy with the bugfix for cli, there is needed slight modification to compile (on quarkus side). |
e1960b1
to
0358ab4
Compare
0358ab4
to
62064df
Compare
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.
lgtm
Should we at least let them know about this? I approved as this is FW, so disabled test inside FW doesn't bother me (we have TS to verify RHBQ), but maybe we should be good citizens? I'll leave decision to you. |
Summary
Update Quarkus version to 3.2.12.Final
Backport helm removal as this was breaking windows CI execution.
Disable CLI test as they are affected by quarkusio/quarkus#37808
Please check the relevant options
run tests
phrase in comment)Checklist: