Skip to content
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 Kubernetes 1.28 release blog article #42381

Merged

Conversation

bradmccoydev
Copy link
Member

@bradmccoydev bradmccoydev commented Aug 4, 2023

Adding Draft Release blog for Kubernetes 1.28 release.

cc Comms team @krol3 @ramrodo

cc Leads @gracenng @leonardpahlke @Atharva-Shinde @sanchita-07 @furkatgofurov7 @mehabhalodiya @Rishit-dagli

Closes issue: #42380

Signed-off-by: Brad McCoy <bradmccoydev@gmail.com>
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 4, 2023
@k8s-ci-robot k8s-ci-robot added the area/blog Issues or PRs related to the Kubernetes Blog subproject label Aug 4, 2023
@k8s-ci-robot k8s-ci-robot requested a review from sftim August 4, 2023 04:52
@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Aug 4, 2023
@netlify
Copy link

netlify bot commented Aug 4, 2023

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 47d9dd5
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/64dba198caffc700089d6bc8
😎 Deploy Preview https://deploy-preview-42381--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.

@sftim
Copy link
Contributor

sftim commented Aug 4, 2023

/hold

We shouldn't merge this until the release is confirmed

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 4, 2023
@sftim
Copy link
Contributor

sftim commented Aug 4, 2023

/retitle [WIP] Add kubernetes 1.28 release blog article

@k8s-ci-robot k8s-ci-robot changed the title add kubernetes 1.28 release blog [WIP] Add kubernetes 1.28 release blog article Aug 4, 2023
@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 Aug 4, 2023
Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

One thing that's good to do:

Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

Some early feedback

content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
bradmccoydev and others added 9 commits August 4, 2023 20:12
Signed-off-by: Brad McCoy <bradmccoydev@gmail.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

We have a lot of “major“ themes. A shorter list would be better (maybe something to raise in the post-release commsn & retrospective).

content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
content/en/blog/_posts/2023-08-04-kubernetes-1.28-blog.md Outdated Show resolved Hide resolved
Copy link
Member

@kerthcet kerthcet left a comment

Choose a reason for hiding this comment

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

From scheduling POV, one of the major theme should be kubernetes/kubernetes#118551, I think it worths to mention. For other big changes, can you recall anything, @alculquicondor All the KEP related PRs haven't been merged yet, like kubernetes/kubernetes#119208, kubernetes/kubernetes#116461, kubernetes/kubernetes#116065

Co-authored-by: Craig Box <craig.box@gmail.com>
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 15, 2023
@k8s-ci-robot k8s-ci-robot requested a review from sftim August 15, 2023 09:59
@sftim sftim dismissed stale reviews from Rishit-dagli and mickeyboxell August 15, 2023 10:37

nits addressed

Co-authored-by: Tim Bannister <tim@scalefactory.com>
@sftim
Copy link
Contributor

sftim commented Aug 15, 2023

/lgtm

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

LGTM label has been added.

Git tree hash: 731689881a269f591b935fae20540e18f5ec5287


We would like to thank the entire release team for the hours spent hard at work to ensure we deliver a solid Kubernetes v1.28 release for our community.

Special thanks to our release lead, Grace Nguyen, for guiding us through a smooth and successful release cycle and facilitating the communication and collaboration between the different release teams taking advantage of her experience as a shadow and lead in these teams. If you would like to learn more about Grace's journey and this release, listen to the [Kubernetes v1.28 release interview podcast](https://craigbox.substack.com/p/the-kubernetes-128-release-interview).
Copy link
Member

Choose a reason for hiding this comment

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

Hi folks, I actually would prefer to not include the podcast in the blog since it's a third-party source. I'll did multiple interviews for this release so would like to remain fair to all of them.

cc @craigbox @sftim @bradmccoydev

Copy link
Contributor

Choose a reason for hiding this comment

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

@craigbox, thoughts?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm fine with either:

  • we link to each interview
  • we omit the interview from the announcement

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree with @gracenng, I don't think it's appropriate to link to a single interview in this release blog and would like to see this dropped.

Copy link
Member

Choose a reason for hiding this comment

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

Please remove this from the PR.

@sftim
Copy link
Contributor

sftim commented Aug 15, 2023

@bradmccoydev I suggest squashing this to 1 commit - 97 commits (soon to be 98?) is a lot. The resulting commit message will be larger than we like if we get GitHub to do a squash.

If you'd like a maintainer to do that squash for you, please reply here.

@reylejano
Copy link
Member

reylejano commented Aug 15, 2023

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: craigbox, reylejano, Rishit-dagli, 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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Co-authored-by: Grace Nguyen <42276283+gracenng@users.noreply.github.com>
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 15, 2023
@k8s-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@k8s-ci-robot k8s-ci-robot requested a review from sftim August 15, 2023 16:02
@saiyam1814
Copy link

In Automatic, retroactive assignment of a default StorageClass graduates to stable
it is

Kubernetes v1.28 is is automatic and always active; the feature has graduated to stable (general availability).

it should be

Kubernetes v1.28 it is automatic and always active; the feature has graduated to stable (general availability).


For instance, if indexed jobs were used as the basis for a suite of long-running integration tests, then each test run would only be able to find a single test failure.


Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

Nit: an extra line in this spot.

@Rishit-dagli
Copy link
Member

/hold cancel

@Rishit-dagli Rishit-dagli removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 15, 2023
@Rishit-dagli Rishit-dagli merged commit 6ca287b into kubernetes:main Aug 15, 2023
@Rishit-dagli
Copy link
Member

In Automatic, retroactive assignment of a default StorageClass graduates to stable it is

Kubernetes v1.28 is is automatic and always active; the feature has graduated to stable (general availability).

it should be

Kubernetes v1.28 it is automatic and always active; the feature has graduated to stable (general availability).

Myabe we could have this as a separate PR to main?

@saiyam1814
Copy link

Yep


This will lower the infrastructure barrier to enforcing customizable policies as well as providing primitives that help the community establish and adhere to the best practices of both K8s and its extensions.

To use [ValidatingAdmissionPolicies](/docs/reference/access-authn-authz/validating-admission-policy/), you need to enable the `admissionregistration.k8s.io/v1beta1` API group in your cluster's control plane.
Copy link
Contributor

Choose a reason for hiding this comment

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

you need to enable both the `admissionregistration.k8s.io/v1beta1` API group and `ValidatingAdmissionPolicy` feature gate in your cluster's control plane.

cc @gracenng Since this info is critical, is it possible to edit the blog to address this?

Copy link
Contributor

Choose a reason for hiding this comment

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

@cici37, please send in a pull request to get that updated.

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 sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.