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

Simplify Jackson configuration #20468

Merged
merged 1 commit into from
Sep 30, 2021
Merged

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Sep 30, 2021

This is done by using the build-time-and-runtime-fixed phase
and also by using Arc's ability of making config root items beans

N.B. This was erroneously done before in #14588 (the reason the PR tests passed was because of the dependency change - this should not have been done), but then it
didn't work properly because of the Kubernetes Client extension
having its dependencies entangled with the Jackson extension.
The problem has since been fixed

This is done by using the build-time-and-runtime-fixed phase
and also by using Arc's ability of making config root items beans

N.B. This was erroneously done before in quarkusio#14588, but then it
didn't work properly because of the Kubernetes Client extension
having its dependencies entangled with the Jackson extension.
The problem has since been fixed
@quarkus-bot quarkus-bot bot added the area/jackson Issues related to Jackson (JSON library) label Sep 30, 2021
@geoand geoand requested a review from mkouba September 30, 2021 07:28
@geoand
Copy link
Contributor Author

geoand commented Sep 30, 2021

The failures have to do with quay.io being down ATM

@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 30, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Sep 30, 2021

Failing Jobs - Building 8f61378

Status Name Step Failures Logs Raw logs
Gradle Tests - JDK 11 Windows Build Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

io.quarkus.gradle.devmode.QuarkusDevDependencyDevModeTest.main line 14 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeTestUtils that uses java.util.function.Supplier, java.util.function.Supplierjava.util.concurrent.atomic.AtomicReference, java.util.concurrent.atomic.AtomicReferencejava.lang.String, java.lang.Stringboolean was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:166)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

@geoand geoand merged commit f787bcf into quarkusio:main Sep 30, 2021
@quarkus-bot quarkus-bot bot added this to the 2.4 - main milestone Sep 30, 2021
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 30, 2021
@geoand geoand deleted the jackson-simplification branch September 30, 2021 13:28
@gsmet
Copy link
Member

gsmet commented Oct 15, 2021

Ah cool, I knew I was doing something right :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jackson Issues related to Jackson (JSON library)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants