-
Notifications
You must be signed in to change notification settings - Fork 40k
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 etcd to v2.0.5 on the master #5801
Conversation
@@ -10,7 +10,7 @@ | |||
# shasum <tar> | |||
# 6. Update this file with new tar version and new hash | |||
|
|||
{% set etcd_version="v2.0.0" %} | |||
{% set etcd_version="v2.0.5" %} | |||
{% set etcd_tar_url="https://storage.googleapis.com/kubernetes-release/etcd/etcd-%s-linux-amd64.tar.gz" | |||
| format(etcd_version) %} | |||
{% set etcd_tar_hash="sha1=b3cd41d1748bf882a58a98c9585fd5849b943811" %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems unlikely that v2.0.5 has the same hash?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Fixed.
b40c1ec
to
fa563d0
Compare
I assume you ran e2es on this? |
I am still testing to make sure that e2e works and I am creating a cluster by hand to make sure everything is fine and to manually inspect that the master is indeed running v2.0.5. Please do not merge until I have verified that everything is OK. Thank you. |
I can do things like |
I am not convinced this is working yet. When I query the Elasticsearch service through the proxy interface I sometimes get:
and sometimes I get the expected result:
@vishh and I were talking earlier about accessing cluster services via the proxy vs. ELB. |
"Only" 3 e2e failures. The usual suspects.
|
I made another cluster and it seems to work fine. I can list pods. Elasticsearch consistently returns the correct value. The Kibana dashboard works.
I propose that we merge this upgrade to etcd. |
@kelseyhightower FYI we are about to upgrade to etcd v2.0.5 and I will also upgrade to the latest go-etcd when I can defeat Godeps. |
This is more testing than most things get. I'm sniffing at Jenkins now. |
Upgrade etcd to v2.0.5 on the master
This changes our Salt setup to install etcd v2.0.5 on the master.
Running tests to verify this actually works.