Skip to content

Conversation

@rytswd
Copy link
Member

@rytswd rytswd commented Mar 28, 2025

This is ready for review, with all the details including logo + theme and webinar.

The blog preview is available with https://deploy-preview-50303--kubernetes-io-main-staging.netlify.app/blog/2025/04/23/kubernetes-v1-33-release/

All the actions below have been complete

  • Ensure webinar link is working (currently a broken link)
    • This is being worked on with CNCF
  • Follow up on the open discussion points (if no response by EOD Monday, we will proceed with what we have now)
    • Closing all the remaining discussion points as of 11 PM BST
  • Word wrap the entire document
  • Squash and clean up the commit history

Note

There was also Google Doc draft which was used for the initial review. It is now read-only, and all the reviews should take place here.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. area/blog Issues or PRs related to the Kubernetes Blog subproject size/S Denotes a PR that changes 10-29 lines, ignoring generated files. language/en Issues or PRs related to English language labels Mar 28, 2025
@rytswd rytswd marked this pull request as draft March 28, 2025 18:28
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 28, 2025
@netlify
Copy link

netlify bot commented Mar 28, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 7264776
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6806dc4c6916d4000864b7b3
😎 Deploy Preview https://deploy-preview-50303--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rytswd rytswd force-pushed the add-release-announcement-blog-draft branch from 12e19ca to 96ff763 Compare April 12, 2025 04:00
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 12, 2025
@rytswd rytswd marked this pull request as ready for review April 12, 2025 04:25
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 12, 2025
@rytswd
Copy link
Member Author

rytswd commented Apr 12, 2025

The release announcement blog is now ready for review!

CC: @npolshakova @katcosgrove @mbianchidev @aibarbetta @aakankshabhende @Udi-Hofesh @sn3hay

@rytswd
Copy link
Member Author

rytswd commented Apr 12, 2025

I just realised there are 3 other items missing, which are not directly tied to alpha/beta/GA features. I will try to work it out over the weekend.

Copy link
Member

@lmktfy lmktfy left a comment

Choose a reason for hiding this comment

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

Partial feedback, focused on the graduations to stable.

Udi-Hofesh

This comment was marked as duplicate.

@rytswd rytswd requested a review from tengqm April 20, 2025 00:37
Copy link
Member

@lmktfy lmktfy left a comment

Choose a reason for hiding this comment

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

Thanks!

/lgtm
I recommend a squash before merging.


This work was done as part of [KEP-1495: Generic data populators](https://kep.k8s.io/1495) led by SIG Storage.

### Always Honor PersistentVolume Reclaim Policy
Copy link
Member

Choose a reason for hiding this comment

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

nit: heading should be sentence case

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh thanks I thought I fixed all, will address this before merge!

Copy link
Member Author

Choose a reason for hiding this comment

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

This is fixed with 472dd6f

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 20, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 551c7e5c6c605762412829145ad9ce01d42e9482

@lmktfy lmktfy dismissed Udi-Hofesh’s stale review April 20, 2025 14:32

This looks good to merge as a draft.

Copy link
Contributor

@graz-dev graz-dev left a comment

Choose a reason for hiding this comment

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

Great piece folks!
That's a very good summary for the release!

/lgtm

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 21, 2025
@k8s-ci-robot k8s-ci-robot requested review from graz-dev and lmktfy April 21, 2025 23:42
@rytswd rytswd force-pushed the add-release-announcement-blog-draft branch from 2fa69d9 to 7264776 Compare April 22, 2025 00:01
@rytswd
Copy link
Member Author

rytswd commented Apr 22, 2025

Thanks to all those who contributed with the review, this PR is finally fully ready to be merged.
As there were dozens of commits, I squashed all and force pushed. The old commits can be found in https://github.com/rytswd/k8s-website/tree/v1.33-release-announcement-backup-branch.

@natalisucks @katcosgrove If you could help approve + merge this PR, that would be much appreciated!
(NOTE: This blog still has the draft: true front matter, and thus it won't be published. I will be creating a separate PR for that removal)

CC: @npolshakova @lmktfy

@rytswd
Copy link
Member Author

rytswd commented Apr 22, 2025

I would like to especially thank all of the v1.33 Comms team, @aibarbetta @aakankshabhende @Udi-Hofesh and @sn3hay for all of their effort into writing up this blog. This happens to be the longest blog in the recent releases (7000+ words, compared to the recent second longest of v1.31 with 5000+ words), and it would have not been possible without their continuous effort and support.

Great work, team! 🥳

@katcosgrove
Copy link
Contributor

/lgtm
/approve

Ship it!

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 22, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 8df225aea7f41c7eeaf7abb0feb3337b33396967

@lmktfy
Copy link
Member

lmktfy commented Apr 22, 2025

/hold cancel

(article is marked draft)

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 22, 2025
@k8s-ci-robot k8s-ci-robot merged commit 1b53071 into kubernetes:main Apr 22, 2025
6 checks passed
@rytswd rytswd deleted the add-release-announcement-blog-draft branch April 22, 2025 10:46
@rytswd
Copy link
Member Author

rytswd commented Apr 22, 2025

I created #50579 as a follow-up to publish this blog

Copy link
Member

@Udi-Hofesh Udi-Hofesh left a comment

Choose a reason for hiding this comment

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

lgtm and my name is spelled correctly :]

Copy link
Member

@aakankshabhende aakankshabhende 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
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: katcosgrove, lmktfy, natalisucks, OBITOONDEADO1MZ

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/blog Issues or PRs related to the Kubernetes Blog subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.