Skip to content
This repository has been archived by the owner on Feb 12, 2020. It is now read-only.

KOGITO-772 - Upgrade Quarkus to version 1.1.0 #78

Merged
merged 1 commit into from
Jan 13, 2020

Conversation

cristianonicolai
Copy link
Contributor

@cristianonicolai cristianonicolai commented Dec 19, 2019

@@ -58,7 +58,7 @@
<version.com.github.javaparser>3.13.10</version.com.github.javaparser>
<version.com.fasterxml.jackson>2.9.10</version.com.fasterxml.jackson>
<version.kubernetes-client>4.4.2</version.kubernetes-client>
<version.okhttp>3.12.0</version.okhttp>
<version.okhttp>3.12.6</version.okhttp>
Copy link
Member

Choose a reason for hiding this comment

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

@cristianonicolai we might need too revert back this one or upgrade the kubernetes-client version also:
https://github.com/fabric8io/kubernetes-client/blob/v4.4.2/pom.xml#L85

But this kubernetes-client must follow Quarkus one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ricardozanini done :)

@cristianonicolai cristianonicolai force-pushed the KOGITO-772 branch 2 times, most recently from 1814b7f to 67f5e86 Compare January 6, 2020 18:02
pom.xml Outdated
@@ -57,8 +57,8 @@
<version.org.eclipse.jdt>3.19.0</version.org.eclipse.jdt>
<version.com.github.javaparser>3.13.10</version.com.github.javaparser>
<version.com.fasterxml.jackson>2.9.10</version.com.fasterxml.jackson>
<version.kubernetes-client>4.4.2</version.kubernetes-client>
<version.okhttp>3.12.0</version.okhttp>
<version.kubernetes-client>4.6.4</version.kubernetes-client>
Copy link
Contributor

Choose a reason for hiding this comment

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

updating kubernetes-client to 4.6.4 requires an update of jackson
see #76
We should put version 4.6.2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@radtriste ack, thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants