-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Announce v1.29 release #43939
Announce v1.29 release #43939
Conversation
Signed-off-by: carolina valencia <krol3@users.noreply.github.com>
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
/hold Do not publish until Kubernetes v1.29 is 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.
Thanks!
Here's some early feedback.
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.
Given the doc is still in an early state, it can be useful to iterate in a hack.md file or somewhere less formal than Github to avoid tracking every addition and revision.
|
||
### Deprecations and removals | ||
|
||
#### Removal of in-tree integrations with cloud providers ([KEP-2395](https://kep.k8s.io/2395)) |
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 pretty big change, thanks for adding it to the deprecations/removal list. I'm wondering if there's more we can do to make this change stand out more since it's a breaking change for some clusters. Can we maybe add a "What's new in sig-cloud-provider?" section at the top that also highlights this? Thoughts @elmiko @sftim (continuing the discussion from #44010)
@krol3, are folks in release comms willing to revise this given the feedback in https://github.com/kubernetes/website/pull/43939/files#r1393368279? |
Hi @sftim please review the new order about the major themes. Thanks :) |
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've added a number of minor comments to be addressed.
Given this needs to ship in a few hours, could we commit the final few suggestions and save everything else for after this is live? |
cc @natalisucks, Mickey's comment above |
@katcosgrove @mickeyboxell I'm fine with committing directly given the timeline – please go ahead and I'll be around to approve |
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Discussed with Priyanka on the release team call and she agreed to close this review.
suggestions have all been applied, so in the interest of this tight timeframe for the release, this is good as is |
LGTM label has been added. Git tree hash: e830587381bf7f6754e534d37d54b9c713dc831d
|
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: reylejano, 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 |
🛑 DO NOT MERGE 🛑 UNTIL KUBERNETES 1.29 IS RELEASED
Release announcement blog article for 1.29
If you prefer to collaborate using hackmd, please use it and put in the comments to include it.