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

Staging 1.5 client #32906

Merged
merged 3 commits into from Sep 23, 2016
Merged

Conversation

caesarxuchao
Copy link
Member

@caesarxuchao caesarxuchao commented Sep 16, 2016

Created the 1.5 folder and remove the 1.4 folder in the staging area in the master branch.

Content of kubernetes/client-go/1.4 will be pulled from the kubernetes/kubernetes 1.4 branch (kubernetes-retired/contrib#1719)


This change is Reviewable

@caesarxuchao caesarxuchao added the sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. label Sep 16, 2016
@k8s-github-robot k8s-github-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. release-note-label-needed labels Sep 16, 2016
@caesarxuchao
Copy link
Member Author

@k8s-bot test this, issue #IGNORE

@k8s-bot
Copy link

k8s-bot commented Sep 19, 2016

GCE e2e build/test passed for commit 74f26cb1f223ab305cad988db4c3d2f6dc07e802.

@lavalamp
Copy link
Member

Can you make the commit in the middle do a rename instead of having a delete & then recreate commit?

Also test is red?

Then lgtm.

@caesarxuchao
Copy link
Member Author

I merged the commit, but github doesn't show them as renaming, maybe it's because too many files are moved.

@lavalamp
Copy link
Member

I think you may have dropped the second commit instead of squashing them, github says only +7 lines right now...

@caesarxuchao
Copy link
Member Author

No, I didn't drop it. Github is broken.

extensions "k8s.io/client-go/1.4/pkg/apis/extensions/v1beta1"
policy "k8s.io/client-go/1.4/pkg/apis/policy/v1alpha1"
"k8s.io/client-go/1.4/pkg/util/intstr"
release_1_4 "k8s.io/client-go/1.5/kubernetes"
Copy link
Contributor

Choose a reason for hiding this comment

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

Bad name for this import.

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice catch. I remove the rename.

@mml
Copy link
Contributor

mml commented Sep 21, 2016

@caesarxuchao Also I think you want release-note-none.

@caesarxuchao caesarxuchao added release-note-none Denotes a PR that doesn't merit a release note. and removed release-note-label-needed labels Sep 21, 2016
@caesarxuchao
Copy link
Member Author

Can I get the label? Thanks.

@mml
Copy link
Contributor

mml commented Sep 21, 2016

@lavalamp git doesn't actually have a way to track file rename/move operations. Instead it does "rename detection" when it tries to summarize a change to you.

LGTM

@mml mml added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 21, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins GCE e2e failed for commit a4e6229.

The magic incantation to run this job again is @k8s-bot gce e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@caesarxuchao
Copy link
Member Author

@k8s-bot e2e test this, issue #33175.

@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 1ef86aa into kubernetes:master Sep 23, 2016
k8s-github-robot pushed a commit that referenced this pull request Sep 27, 2016
Automatic merge from submit-queue

Copy auth plugin to client-go repo

client-go doesn't copy the [auth plugin](https://github.com/kubernetes/kubernetes/blob/master/plugin/pkg/client/auth/plugins.go). This causes user cannot access cluster run by GKE. User will see error "No Auth Provider found for name gcp".

This PR fixes this issue. It's marked as WIP because I'll need to rebase after #32906 gets merged. Also, the fix needs to be cherry-picked into 1.4 branch to update client-go/1.4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants