Skip to content
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

Ignore non-existing class and resource paths during JVM test #28186

Merged
merged 1 commit into from
Sep 29, 2022

Conversation

AdlerFleurant
Copy link
Contributor

Some resource and class locations may not exist when resolving paths.
We now ensure that these non-existing paths are not added to the application root.

@AdlerFleurant
Copy link
Contributor Author

This may also fix #26013

@quarkus-bot

This comment has been minimized.

@AdlerFleurant
Copy link
Contributor Author

Do we need to rerun this or should we investigate?

Some resource and class locations may not exist when resolving paths.
We now ensure that these non-existing path are not added to the application root.

Signed-off-by: Adler Fleurant <[email protected]>
@gsmet
Copy link
Member

gsmet commented Sep 28, 2022

Yeah no need to push again when it's like that. Usually it's just a flaky test. We will deal with it ourselves and ping you if we think it's related to your work.

We are currently working on improving the situation with flaky tests.

@gsmet gsmet requested a review from aloubyansky September 28, 2022 14:41
@quarkus-bot

This comment has been minimized.

@quarkus-bot
Copy link

quarkus-bot bot commented Sep 29, 2022

Failing Jobs - Building 33f1e6c

Status Name Step Failures Logs Raw logs
✔️ JVM Tests - JDK 11
✔️ JVM Tests - JDK 17
JVM Tests - JDK 17 MacOS M1 Set up runner ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 18

Copy link
Member

@aloubyansky aloubyansky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aloubyansky aloubyansky merged commit 9367382 into quarkusio:main Sep 29, 2022
@quarkus-bot quarkus-bot bot added this to the 2.14 - main milestone Sep 29, 2022
@gsmet gsmet modified the milestones: 2.14 - main, 2.13.1.Final Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants