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

No PetSet client in client/unversioned #23

Closed
wants to merge 1 commit into from

Conversation

ironcladlou
Copy link

@ironcladlou ironcladlou commented Aug 18, 2016

Missed pick of kubernetes#29655. Next, origin will need to specify b81a8de in Godeps.json which should have been included with openshift/origin@eb0dc79.

@ironcladlou
Copy link
Author

@deads2k
Copy link

deads2k commented Aug 18, 2016

@ironcladlou does this already existing in openshift/origin? If not, open the pull there first. Once we get a clean merge in openshift/origin, we can do a bulk pick down into openshift/kubernetes.

We never want to have a state where openshift/kubernetes can't be used to build openshift/origin and right now the only way to do that is "origin first".

@ncdc
Copy link

ncdc commented Aug 18, 2016

This came in origin as part of openshift/origin#9972

@ironcladlou
Copy link
Author

ironcladlou commented Aug 18, 2016

Actually, does it even need to be picked into this fork? As long as the commit exists upstream and hack/godep-restore.sh fetches both the upstream and our fork remotes (which it does), just referencing the correct commit in Godeps.json should be enough. Agree?

@ironcladlou
Copy link
Author

Nevermind, this isn't necessary.

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

Successfully merging this pull request may close these issues.

None yet

4 participants