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

Update vendor files to point to k8s 1.12.0 #1009

Merged
merged 1 commit into from
Oct 5, 2018

Conversation

xing-yang
Copy link
Contributor

@xing-yang xing-yang commented Oct 1, 2018

This PR updates dependency files to point to k8s 1.12.0.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Oct 1, 2018
@xing-yang
Copy link
Contributor Author

@verult Please take a look. Thanks.

@verult
Copy link
Contributor

verult commented Oct 1, 2018

Should we wait until the kubernetes-1.12.0 release comes out? The release-1.12 branch is going to keep changing

@xing-yang
Copy link
Contributor Author

@verult What will release-1.12 branch change to? 1.12.1, 1.12.2,...1.12.x?
Not sure when kubernetes-1.12.0 will be released for api, apimachinery, etc.

Gopkg.toml Outdated
@@ -64,23 +64,23 @@ ignored = [

[[constraint]]
name = "k8s.io/api"
version = "kubernetes-1.12.0-beta.1"
branch = "release-1.12"
Copy link
Contributor

Choose a reason for hiding this comment

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

To guarantee all repos are in sync and compatible at e.g. VERSION=1.12.0-rc.2, use the same tag "kubernetes-$VERSION" for all k8s.io/* repos and tag "v-$VERSION" for k8s.io/kubernetes. The repos will always follow this naming scheme and have the same tags e.g. "kubernetes-1.12.0-rc.2" available at the same time

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, I can use the kubernetes-1.12.0-rc.2 for everything to be consistent.

@xing-yang xing-yang changed the title Update vendor files to point to k8s 1.12.0 release Update vendor files to point to k8s 1.12.0 RC2 Oct 2, 2018
@verult
Copy link
Contributor

verult commented Oct 2, 2018

The contents of release-1.12 is going to keep changing and not always aligned with specific releases. Yeah I agree at the very least we should use a tagged version.

Do we have to update the dependency versions now? Can we wait until 1.12.0 is out for api/apimachinery etc? We want to use that version in the end anyway right?

@xing-yang
Copy link
Contributor Author

Let's discuss about this in tomorrow's meeting and decide what to do. Thanks.

@xing-yang
Copy link
Contributor Author

Discussed about this at today's meeting. We can wait for the new tag.

This PR updates dependency files to point to k8s 1.12.0.
@xing-yang xing-yang changed the title Update vendor files to point to k8s 1.12.0 RC2 Update vendor files to point to k8s 1.12.0 Oct 5, 2018
@xing-yang
Copy link
Contributor Author

Updated with tag kubernetes-1.12.0 which is available now.

@xing-yang
Copy link
Contributor Author

@wongma7 @verult Please take a look.

@wongma7
Copy link
Contributor

wongma7 commented Oct 5, 2018

/lgtm
thank you

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 5, 2018
@wongma7 wongma7 merged commit 193fb39 into kubernetes-retired:master Oct 5, 2018
@xing-yang
Copy link
Contributor Author

@wongma7 Thanks for merging this! Can you create a tag for external-storage so we can reference it in external-provisioner?

@wongma7
Copy link
Contributor

wongma7 commented Oct 5, 2018

@xing-yang created tag v5.2.0

@xing-yang
Copy link
Contributor Author

@wongma7 Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. 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.

4 participants