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.

Sign up for GitHub

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

Jump to bottom

Provide meaningful error message when updating Quarkus in dev-mode #22151

Merged
merged 1 commit into from
Dec 13, 2021
Merged

Provide meaningful error message when updating Quarkus in dev-mode #22151

merged 1 commit into from
Dec 13, 2021

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Dec 13, 2021

Closes: #22105

@geoand geoand mentioned this pull request Dec 13, 2021
Closed
@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Dec 13, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Dec 13, 2021

Failing Jobs - Building 7c758c7

Status Name Step Failures Logs Raw logs
✔️ JVM Tests - JDK 11
✖ JVM Tests - JDK 17 Build Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/opentelemetry/opentelemetry/deployment 
! Skipped: extensions/opentelemetry/opentelemetry-exporter-jaeger/deployment extensions/opentelemetry/opentelemetry-exporter-otlp/deployment integration-tests/opentelemetry and 2 more

📦 extensions/opentelemetry/opentelemetry/deployment

✖ io.quarkus.opentelemetry.deployment.RestClientOpenTelemetryTest.spanNameWithoutQueryString line 88 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a io.quarkus.opentelemetry.deployment.TestSpanExporter expected: <2> but was: <1> within 30 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:164)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)

@geoand geoand merged commit ab382fe into quarkusio:main Dec 13, 2021
@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Dec 13, 2021
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Dec 13, 2021
@gsmet gsmet modified the milestones: 2.7 - main, 2.6.0.Final Dec 13, 2021
@geoand geoand deleted the #22105 branch December 14, 2021 05:56
@gsmet gsmet modified the milestones: 2.6.0.Final, 2.5.3.Final Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsmet gsmet gsmet approved these changes

@aloubyansky aloubyansky aloubyansky approved these changes

Assignees
No one assigned
Labels
area/core
Projects
None yet
Milestone
2.5.3.Final
Development

Successfully merging this pull request may close these issues.

NPE while live updating quarkus from 2.5.0 to 2.5.1 in RuntimeUpdatesProcessor
4 participants
@geoand @aloubyansky @gsmet @gastaldi