Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

brendandburns
Copy link
Contributor

@dchen1107
Copy link
Member

Are we comfortable to upgrade etcd server to a new version now?

@thockin
Copy link
Member

thockin commented Jun 19, 2015

The risk of this falling over is scary - what's the reward?

@lavalamp
Copy link
Contributor

Very scary, esp since I have no way of telling what source was sitting on your computer when you pushed the button.

@davidopp
Copy link
Contributor

Very scary, esp since I have no way of telling what source was sitting on your computer when you pushed the button.

Not disagreeing, but can you explain? Isn't the makefile just building a Docker container from a binary it fetches? How is source code sitting on his computer related?

@lavalamp
Copy link
Contributor

The makefile builds source.

@dchen1107
Copy link
Member

For issue #10122, @cjcullen and I discussed, the current proposed solution is not starting any pods / containers until network configuration is done. In this case, shouldn't no rush to upgrade to a new etcd version?

@alex-mohr alex-mohr added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 22, 2015
@alex-mohr
Copy link
Contributor

etcd 2.0.9 was also built from someone's desktop. After chatting with a few people and looking at the CLs pulled into 2.0.12 -- especially fixes for bugs that cause an etcd install to become semi-permanently unhealthy -- it seems like a reasonable approach to switch to it as the least-worst option.

And if the release-of-etcd process is scary, we should figure out a sustainable long term approach but not block this particular PR on it.

@dchen1107
Copy link
Member

LGTM based on our offline discussion.

@dchen1107
Copy link
Member

I am going to merge this one for now, so that @zmerlynn can cut 0.19.3 to include this one. @zmerlynn please make sure you cut 0.19.3 is solely based on 0.19.1 + this pr. Thanks!

@zmerlynn
Copy link
Member

On it.

@alex-mohr
Copy link
Contributor

... and apologies to those not involved in offline discussions.

Note that if we find issues with 2.0.12 in the RC validation process, we can roll it back.

@dchen1107
Copy link
Member

Ok to test.

@zmerlynn
Copy link
Member

I'm going to go ahead and merge this - we've had a lot of internal soak time with these bits from a private build.

zmerlynn added a commit that referenced this pull request Jun 22, 2015
@zmerlynn zmerlynn merged commit 1559fa8 into kubernetes:master Jun 22, 2015
This was referenced Jun 22, 2015
sttts added a commit to mesosphere-backup/kubernetes that referenced this pull request Jun 24, 2015
sttts added a commit to mesosphere-backup/kubernetes that referenced this pull request Jul 2, 2015
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants