-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Add etcd-manager certificate expiration advisory #9030
Add etcd-manager certificate expiration advisory #9030
Conversation
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.
I would also add the workaround as a ## Workaround at the end.
#8959 (comment)
The reason for also adding the workaround is that it can be done without a |
1f7d436
to
00e7659
Compare
I added the workaround, mentioning that it will only delay the issue. Also updated the kops 1.18 versions. /hold for more review and for the new kops versions to be released |
00e7659
to
3ed83e1
Compare
5445837
to
6e43e78
Compare
6e43e78
to
d050dcd
Compare
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.
/hold
/lgtm
This looks great @rifelpet thanks! I left some minor edits. Feel free to take or leave them
holding to give a chance for others to see
d050dcd
to
cdbf1e8
Compare
cdbf1e8
to
6130a3f
Compare
/lgtm |
6130a3f
to
e8dc780
Compare
/test pull-kops-bazel-test |
The etcd-manager version is set automatically based on the Kops version. | ||
These Kops versions are affected: | ||
|
||
* Kops 1.10.0-alpha.1 through 1.16.1 |
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.
Safe versions are going to be 1.15.3, 1.16.2, 1.17.0-beta.2, 1.18.0-alpha.3
Releases of 1.15.3 and 1.16.2 are staged but not published (please take a look at the message). I figured we should probably do them all at once...
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.
ok, i updated the list of affected versions and the list of fixed versions in the solutions section below.
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.
I updated the release drafts to link to kops.sigs.k8s.io, and confirmed this page looks good on netlify.
e3d51b8
to
9e2355f
Compare
9e2355f
to
836545d
Compare
Thanks - I think it's go time! /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb, rifelpet 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 |
A few notes:
I'd like this to be throughly reviewed before we merge since people will start seeing it and linking to it.
/cc @justinsb