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

Include the artifact in the bean discovery mode warning #31379

Merged
merged 1 commit into from
Feb 27, 2023

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Feb 23, 2023

Otherwise it is not possible to know which dependency contain the problematic beans.xml.

Related to #31378.

Otherwise it is not possible to know which dependency contain the
problematic beans.xml.

Related to quarkusio#31378.
@gsmet gsmet requested a review from Ladicek February 23, 2023 17:40
@quarkus-bot quarkus-bot bot added the area/arc Issue related to ARC (dependency injection) label Feb 23, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Feb 23, 2023

Failing Jobs - Building ae647a6

Status Name Step Failures Logs Raw logs
✔️ Devtools Tests - JDK 11
Devtools Tests - JDK 11 Windows Build Failures Logs Raw logs
✔️ Devtools Tests - JDK 17
✔️ JVM Tests - JDK 11
JVM Tests - JDK 11 Windows Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 17
✔️ JVM Tests - JDK 19
Maven Tests - JDK 11 Build Failures Logs Raw logs
Maven Tests - JDK 11 Windows Build Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Devtools Tests - JDK 11 Windows #

- Failing: integration-tests/devtools 

📦 integration-tests/devtools

io.quarkus.devtools.commands.CreateProjectTest.createMultipleTimes line 254 - More details - Source on GitHub

java.util.concurrent.TimeoutException: createMultipleTimes() timed out after 3 seconds
	at org.junit.jupiter.engine.extension.TimeoutExceptionFactory.create(TimeoutExceptionFactory.java:29)
	at org.junit.jupiter.engine.extension.SameThreadTimeoutInvocation.proceed(SameThreadTimeoutInvocation.java:58)

⚙️ Maven Tests - JDK 11 #

- Failing: integration-tests/maven 

📦 integration-tests/maven

io.quarkus.maven.it.BuildIT.testClassLoaderLinkageError line 102 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: 0

io.quarkus.maven.it.BuildIT.testClassLoaderLinkageError line 102 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: 0

⚙️ Maven Tests - JDK 11 Windows #

📦 integration-tests/maven

io.quarkus.maven.it.BuildIT.testClassLoaderLinkageError line 102 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: 0

io.quarkus.maven.it.BuildIT.testClassLoaderLinkageError line 102 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: 0

Copy link
Contributor

@Ladicek Ladicek left a comment

Choose a reason for hiding this comment

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

Ouch! I thought pathVisit.getPath() will include the full path, not just inside the archive; sorry for not verifying that assumption.

@gsmet
Copy link
Member Author

gsmet commented Feb 27, 2023

@Ladicek no worries, it's easy to miss :).

@gsmet gsmet merged commit c3a2256 into quarkusio:main Feb 27, 2023
@quarkus-bot quarkus-bot bot added this to the 3.0 - main milestone Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/arc Issue related to ARC (dependency injection)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants