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

Upgrade Kubernetes client to 2.2.13. #230

Merged
merged 2 commits into from
Apr 14, 2017

Conversation

mccheah
Copy link

@mccheah mccheah commented Apr 11, 2017

Closes #229.

Copy link

@ash211 ash211 left a comment

Choose a reason for hiding this comment

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

I took a skim through fabric8io/kubernetes-client@v2.2.1...v2.2.13 and didn't see anything immensely scary.

@iyanuobidele @foxish any concerns before merging tomorrow?

@foxish
Copy link
Member

foxish commented Apr 13, 2017

Seems fine. None of it affects the features we're using by the looks of it. I found periodicWatchUntilReady in one of the changes. We may be able to offload some of our work to the library by using this method perhaps? @mccheah

@mccheah
Copy link
Author

mccheah commented Apr 14, 2017

Let's look at that separately - I'm going to merge this as is.

@mccheah mccheah merged commit 0b0fb6f into branch-2.1-kubernetes Apr 14, 2017
@ash211 ash211 deleted the upgrade-kubernetes-client branch April 14, 2017 02:39
foxish pushed a commit that referenced this pull request Jul 24, 2017
ifilonenko pushed a commit to ifilonenko/spark that referenced this pull request Feb 26, 2019
puneetloya pushed a commit to puneetloya/spark that referenced this pull request Mar 11, 2019
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.

3 participants