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

Upgrade kubernetes to V1.11.0 #2989

Closed
wants to merge 1 commit into from
Closed

Upgrade kubernetes to V1.11.0 #2989

wants to merge 1 commit into from

Conversation

riverzhang
Copy link
Contributor

No description provided.

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jul 12, 2018
@riverzhang
Copy link
Contributor Author

ci check this

@NicolasT
Copy link
Contributor

What about #2947 ?

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 12, 2018
@riverzhang riverzhang closed this Jul 12, 2018
@riverzhang riverzhang reopened this Jul 12, 2018
.gitlab-ci.yml Outdated
@@ -93,7 +93,7 @@ before_script:
# Check out latest tag if testing upgrade
# Uncomment when gitlab kubespray repo has tags
#- test "${UPGRADE_TEST}" != "false" && git fetch --all && git checkout $(git describe --tags $(git rev-list --tags --max-count=1))
- test "${UPGRADE_TEST}" != "false" && git checkout f7d52564aad2ff8e337634951beb4a881c0e8aa6
- test "${UPGRADE_TEST}" != "false" && git checkout master

Choose a reason for hiding this comment

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

This won't make sense once this PR lands in master. You would be upgrading from 1.11 -> 1.11.

@riverzhang riverzhang closed this Jul 12, 2018
@riverzhang riverzhang reopened this Jul 12, 2018
README.md Outdated
@@ -105,6 +105,7 @@ Supported Components
- [cephfs-provisioner](https://github.com/kubernetes-incubator/external-storage) v1.1.0-k8s1.10
- [cert-manager](https://github.com/jetstack/cert-manager) v0.3.2
- [ingress-nginx](https://github.com/kubernetes/ingress-nginx) v0.16.2
- [coredns](https://github.com/coredns/coredns) v1.1.2
Copy link
Member

Choose a reason for hiding this comment

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

the right version is v1.1.3 : kubernetes/kubernetes#64258

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks

@@ -112,7 +112,7 @@ kube_apiserver_insecure_port: 8080 # (http)

# Kube-proxy proxyMode configuration.
# Can be ipvs, iptables
kube_proxy_mode: ipvs
kube_proxy_mode: iptables

Choose a reason for hiding this comment

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

seems odd when you are removing istio.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, test install failed with ipvs.

Copy link
Contributor

Choose a reason for hiding this comment

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

We didn't run in any issues yet running kube-proxy using ipvs with Kubespray master (so, Kubernetes 1.10.something), so this seems fishy...

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jul 13, 2018
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 13, 2018
@riverzhang riverzhang closed this Jul 13, 2018
@riverzhang riverzhang reopened this Jul 13, 2018
@riverzhang riverzhang closed this Jul 13, 2018
@riverzhang riverzhang reopened this Jul 13, 2018
@riverzhang riverzhang closed this Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants