-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
WIP: Add doc for CVE-2017-14491 #3534
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Assign the PR to them by writing The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
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.
Looks great! Thanks @justinsb
docs/advisories/cve_2017_14491.md
Outdated
|
||
## Fixed in kops versions: | ||
|
||
Not yet released |
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.
May want to link to the PR for future context: #3511
b9fc58c
to
d588112
Compare
Sorry @mikesplain - I just revised it to the minimal fix and force-pushed - I am a bad person. I will link the PR - good suggestion! |
Are the instructions for 1.5 k8s and 1.6 k8s different? |
Note - these instructions don't work for all versions. If you have applied them, please double-check that your kube-dns pods are healthy. I made a mistake on one of my clusters and conflated the 1.5 and the 1.6/1.7 instructions. If you apply the change, make double sure that |
Should the image name in the hotfix be |
@justinsb PR needs rebase |
Closing because I pr'ed the doc per Justin |
No description provided.