-
Notifications
You must be signed in to change notification settings - Fork 34
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 several disabled tests, remove others #2191
base: main
Are you sure you want to change the base?
Conversation
f43d1a2
to
380d155
Compare
The issue, which caused the test to be disabled were solved, but the test were failing due to the lack of RAM. Adding 1G solved that porblem
baf3bef
to
0cad455
Compare
@fedinskiy formatting check didn't pass Several This PR seems to be suitable for "run tests" phrase |
0cad455
to
7ab93ac
Compare
@rsvoboda |
You need to run them on your own and you should as you are enabling tests specifically for that platform |
@fedinskiy I don't need a proof (link, job run etc.), but I need you to check that enabled tests will suceed on Aarch64, could you check please? |
Following jobs contain at least one flaky test:
Run summary: https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/11839134584?pr=2191 Flaky tests:
Failure stacktrace
|
run tests |
Changes LGTM, thank you. Of enabled Aarch64, following needs to be checked:
Feel free to merge it when you have done so. I have triggered OCP run, let's recheck its results as well please. |
7ab93ac
to
c9dcc88
Compare
Brotli: this works on ARM: |
Following jobs contain at least one flaky test:
Run summary: https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/11855572203?pr=2191 Flaky tests:
Failure stacktrace
|
run tests |
Summary
Please select the relevant options.
run tests
phrase in comment)Checklist: