Enable tests with Quarkus OpenShift extension as upstream issue is fixed #1106
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
run tests |
michalvavrik
changed the title
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
4 times, most recently
from
d8c55ae
to
a434bbb
Compare
michalvavrik
changed the title
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
a434bbb
to
c236ec5
Compare
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
c236ec5
to
bfd76bf
Compare
|
run tests |
|
@michalvavrik what are your plans with this PR? |
|
@rsvoboda problem is that all tests I tried locally (in OC are green), I think there are some params that CI is using that I didn't. I'll proceed on this today, but I have a lot of work on daily failures. |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
bfd76bf
to
e345cda
Compare
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
e345cda
to
574cc62
Compare
|
run tests |
|
Opened quarkusio/quarkus#32193, also CI was adjusted which fixed some of issues. |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
574cc62
to
a3087f0
Compare
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
a3087f0
to
527f757
Compare
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
527f757
to
1155289
Compare
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
1155289
to
d9f0b68
Compare
|
run tests |
michalvavrik
force-pushed
the
feature/enable-tests-using-oc-extension
branch
from
d9f0b68
to
473fed6
Compare
|
run tests |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
OperatorOpenShiftInfinispanCountersITas it sometimes expects 200 status and receives 503, flaky method changes, so I disabled whole classquay.io/quarkus/ubi-quarkus-native-s2iis obsolete and we need to useubi-quarkus-graalvmce-s2i- Quarkus S2I native binary can't be used with OpenShift build configuration quarkusio/quarkus#30829 (comment) I think I changed it as part of debugging, not sure if everything works correctly without this change, but we need to change it anywayI'll investigate disabled tests next week, but this PR is already WIP for quite long time, let's get it in.
Please select the relevant options.
run testsphrase in comment)Checklist: