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.

Sign up for GitHub

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

Jump to bottom

Bump etcd server patch version to 3.2.16 #59836

Merged
merged 1 commit into from
Mar 6, 2018
Merged

Bump etcd server patch version to 3.2.16 #59836

merged 1 commit into from
Mar 6, 2018

Conversation

jpbetz
Copy link
Contributor

@jpbetz jpbetz commented Feb 13, 2018 •
edited

etcd 3.2.16 contains a critical fix for HA clusters: etcd-io/etcd#9281

Also, update newly added tests to use REGISTRY make variable.

Release note:

Upgrade the default etcd server version to 3.2.16

@k8s-ci-robot k8s-ci-robot added do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note 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. labels Feb 13, 2018
@jpbetz jpbetz added this to the v1.10 milestone Feb 13, 2018
@jpbetz jpbetz self-assigned this Feb 13, 2018
@jpbetz jpbetz added sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. release-note-none Denotes a PR that doesn't merit a release note. labels Feb 13, 2018
@mikedanese
Copy link
Member

Does this actually deploy the never version? We need to update other places too, such as: https://github.com/kubernetes/kubernetes/blob/master/build/root/WORKSPACE#L22

@jpbetz
Copy link
Contributor Author

jpbetz commented Feb 26, 2018

@mikedanese Thanks, yes. I've run a full search/replace. PTAL.

cheftako
cheftako approved these changes Feb 26, 2018
View reviewed changes
Copy link
Member

@cheftako cheftako left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 26, 2018
@jpbetz
Copy link
Contributor Author

jpbetz commented Feb 26, 2018

/retest

3 similar comments
@jpbetz
Copy link
Contributor Author

jpbetz commented Feb 27, 2018

/retest

@jpbetz
Copy link
Contributor Author

jpbetz commented Feb 27, 2018

/retest

@jpbetz
Copy link
Contributor Author

jpbetz commented Feb 28, 2018

/retest

@jpbetz
Copy link
Contributor Author

jpbetz commented Mar 1, 2018

Published gcr.io/google-containers/etcd:3.1.16 for all architecture.

/retest

@jpbetz
Copy link
Contributor Author

jpbetz commented Mar 1, 2018

/retest

@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-none Denotes a PR that doesn't merit a release note. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Mar 1, 2018
@jpbetz jpbetz force-pushed the etcd-3.2.16-patch-upgrade branch from 26d3916 to 8edd0fc Compare March 5, 2018 19:19
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 5, 2018
@jpbetz
Copy link
Contributor Author

jpbetz commented Mar 5, 2018

Rebased

jennybuckley
jennybuckley reviewed Mar 5, 2018
View reviewed changes
etcd-v3.2.16-linux-amd64.tar.gz Outdated
@@ -0,0 +1 @@
<html><body>You are being <a href="https://github-production-release-asset-2e65be.s3.amazonaws.com/11225014/88ad2dac-0ffb-11e8-9aa6-eb76dc45bd80?X-Amz-Algorithm=AWS4-HMAC-SHA256&amp;X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20180226%2Fus-east-1%2Fs3%2Faws4_request&amp;X-Amz-Date=20180226T202018Z&amp;X-Amz-Expires=300&amp;X-Amz-Signature=4eb33831fb5b04ca563f484d37e130e123efabd97d5a6e1f63e551ff6a35d02c&amp;X-Amz-SignedHeaders=host&amp;actor_id=0&amp;response-content-disposition=attachment%3B%20filename%3Detcd-v3.2.16-linux-amd64.tar.gz&amp;response-content-type=application%2Foctet-stream">redirected</a>.</body></html>
Copy link

Choose a reason for hiding this comment

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

Why is this file being added?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That shouldn't be there. Removing.

@wenjiaswe
Copy link
Contributor

@jpbetz Last time, when you update etcd server version to 3.2.14, you have changes in staging/src/k8s.io/apiextensions-apiserver/artifacts/example/rc.yaml and staging/src/k8s.io/sample-apiserver/artifacts/example/rc.yaml to change the etcd image from 3.1.10 to 3.2.14, is there any reason that they don't need to be changed this when updating to 3.2.16?

@jpbetz jpbetz force-pushed the etcd-3.2.16-patch-upgrade branch from 8edd0fc to 26e7956 Compare March 5, 2018 20:28
@jpbetz
Copy link
Contributor Author

jpbetz commented Mar 5, 2018

Good catch @wenjiaswe. My sed command filtered those directories out. I've updated the PR to include both rc.yaml files. We'll need to reliable, automated way of performing these version bumps. The current approach is far to labor intensive. I'll open an issue and start thinking about it.

@jpbetz jpbetz mentioned this pull request Mar 5, 2018
Closed
@wenjiaswe
Copy link
Contributor

LGTM

@jpbetz jpbetz force-pushed the etcd-3.2.16-patch-upgrade branch from 26e7956 to 04c6d0a Compare March 5, 2018 21:59
@jennybuckley
Copy link

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 5, 2018
@lavalamp
Copy link
Member

lavalamp commented Mar 5, 2018

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cheftako, jennybuckley, jpbetz, lavalamp

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 5, 2018
@dims
Copy link
Member

dims commented Mar 6, 2018

looks like this has all approvals/lgtms but needs yet another label. applying now ... please holler if i got this wrong

/status approved-for-milestone

@k8s-github-robot
Copy link

[MILESTONENOTIFIER] Milestone Pull Request: Up-to-date for process

@cheftako @jennybuckley @jpbetz

Pull Request Labels
  • sig/api-machinery: Pull Request will be escalated to these SIGs if needed.
  • priority/important-soon: Escalate to the pull request owners and SIG owner; move out of milestone after several unsuccessful escalation attempts.
  • kind/feature: New functionality.
Help

@k8s-github-robot
Copy link

/test all [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions here.

@k8s-github-robot k8s-github-robot merged commit 9aae9b5 into kubernetes:master Mar 6, 2018
@jpbetz jpbetz mentioned this pull request Mar 14, 2018
Merged
k8s-github-robot pushed a commit that referenced this pull request Apr 12, 2018
Merge pull request #61198 from jpbetz/etcd-3.2-upgrade-reattempt
9816b43 
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Update default etcd server to 3.2 for kubernetes 1.11

Repply #59836 but with latest etcd 3.2 patch version (3.2.18 which includes mvcc fix and leader election timeout fix) and default `--snapshot-count` to 10k to resolve performance regression in previous etcd 3.2 server upgrade attempt (#60589 (comment)).

See #60589 (comment) for details on the root cause of the performance regression and scalability test results of setting `--snapshot-count` to 10k.

```release-note
Upgrade the default etcd server version to 3.2.18
```
@gyuho @shyamjvs @jdumars @timothysc
@wenjiaswe wenjiaswe mentioned this pull request Nov 8, 2018
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jennybuckley jennybuckley jennybuckley left review comments

@cheftako cheftako cheftako approved these changes

@mml mml Awaiting requested review from mml

@k8s-mirror-api-machinery-api-reviews k8s-mirror-api-machinery-api-reviews Awaiting requested review from k8s-mirror-api-machinery-api-reviews

@mikedanese mikedanese Awaiting requested review from mikedanese

@zmerlynn zmerlynn Awaiting requested review from zmerlynn

Assignees

@jpbetz jpbetz

@cheftako cheftako

@jennybuckley jennybuckley

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/platform/gce cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
v1.10
Development

Successfully merging this pull request may close these issues.

None yet
10 participants
@jpbetz @mikedanese @thockin @wenjiaswe @jennybuckley @lavalamp @k8s-ci-robot @dims @k8s-github-robot @cheftako