-
Notifications
You must be signed in to change notification settings - Fork 14.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
Feature blog 1.30: user namespaces #45354
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
1e42da9
to
09e6d96
Compare
howdy @rata, I am your assigned 1.30 Release comms shadow and was hoping to help you along with deadlines for this blog. It looks like it is currently a work in progress. |
09e6d96
to
964fb6f
Compare
964fb6f
to
6f577c4
Compare
@Checksumz thanks! This is ready for review |
6f577c4
to
c78ea29
Compare
c78ea29
to
7df68a6
Compare
b11e77c
to
0931de4
Compare
46d9f43
to
33ef483
Compare
@nate-double-u @Checksumz thanks, applied all the suggestions. PTAL! :) |
Thank you @rata. |
If this blog PR is to be merged after the 1.30 release, this should target the |
33ef483
to
a88eefb
Compare
@reylejano thanks for catching that, this is based on (and against) main now. |
It's 9pm here and I'm about to leave, if more changes are needed I hope @giuseppe or @saschagrunert can do them while I'm away in the next few days. |
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
I have some bandwidth today, I'll be reviewing |
/lgtm |
LGTM label has been added. Git tree hash: dd88c5ff781a7332470cb5701d20141ce13ac565
|
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.
/approve
🛑 Do not merge until Kubernetes v1.30 is released 🛑
the exploit, which works without user namespaces, **is mitigated when user | ||
namespaces are in use.** | ||
|
||
{{< youtube id="07y5bl5UDdA" title="Mitigation of CVE-2024-21626 on Kubernetes by enabling User Namespace">}} |
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.
This is a bit large, and title title is slightly off
{{< youtube id="07y5bl5UDdA" title="Mitigation of CVE-2024-21626 on Kubernetes by enabling User Namespace">}} | |
{{< youtube id="07y5bl5UDdA" title="Mitigation of CVE-2024-21626 on Kubernetes by enabling User Namespace support" class="youtube-quote-sm" >}} |
What I recommend: leave this as-is until the release day, then once this has merged (even before publication), send in a follow-up PR.
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.
This is a bit large, and title title is slightly off
What I recommend: leave this as-is until the release day, then once this has merged (even before publication), send in a follow-up PR.
Hi @giuseppe & @saschagrunert, since @rata is on holidays will you be able to address this suggestion ?
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.
Applied the suggestion. 👍
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: matthyx, mrunalp, saschagrunert, sftim 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 |
Signed-off-by: Rodrigo Campos <rodrigoca@microsoft.com> Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
a88eefb
to
052cf7e
Compare
/lgtm |
LGTM label has been added. Git tree hash: 023983989293b510eadbd89edf49d3d3b395dca5
|
/unhold |
Blog post about user namespace migrating to beta in 1.30.
cc @giuseppe