Skip to content
This repository has been archived by the owner on Oct 26, 2021. It is now read-only.

Upgrade io.kubernetes client version to v9.0 #14

Merged
merged 2 commits into from
Sep 29, 2020

Conversation

whynowy
Copy link
Contributor

@whynowy whynowy commented Sep 21, 2020

No description provided.

@CLAassistant
Copy link

CLAassistant commented Sep 21, 2020

CLA assistant check
All committers have signed the CLA.

@whynowy
Copy link
Contributor Author

whynowy commented Sep 21, 2020

@alexec

@crenshaw-dev
Copy link

@ddseapy, does this solve #12? The namespace changes look familiar.

@ddseapy
Copy link

ddseapy commented Sep 21, 2020

no. kubernetes client version upgrade is a pre-requisite for #12, but not sufficient.

Makefile Outdated
--generate-alias-as-model
# add the io.kubernetes:java-client to the deps
sed 's/<dependencies>/<dependencies><dependency><groupId>io.kubernetes<\/groupId><artifactId>client-java<\/artifactId><version>5.0.0<\/version><\/dependency>/g' pom.xml > tmp && mv tmp pom.xml
sed 's/<dependencies>/<dependencies><dependency><groupId>io.kubernetes<\/groupId><artifactId>client-java<\/artifactId><version>9.0.0<\/version><\/dependency>/g' pom.xml > tmp && mv tmp pom.xml
Copy link

Choose a reason for hiding this comment

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

If too soon for 10.0.0 which came out a few days ago, can we bump to 9.0.2 to get the bugfixes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Makes sense.

@whynowy whynowy changed the title Upgrade io.kubernetes client version to v9.0.0 Upgrade io.kubernetes client version to v9.0 Sep 21, 2020
@ddseapy
Copy link

ddseapy commented Sep 21, 2020

lgtm

@alexec alexec merged commit 786209b into argoproj-labs:master Sep 29, 2020
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.

5 participants