Skip to content

Conversation

sreeram-venkitesh
Copy link
Member

Description

Issue

Docs PR for kubernetes/enhancements#4818 GA graduation in v1.34.

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jul 3, 2025
@sreeram-venkitesh
Copy link
Member Author

/sig node

@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. language/en Issues or PRs related to English language sig/node Categorizes an issue or PR as relevant to SIG Node. labels Jul 3, 2025
Copy link

netlify bot commented Jul 3, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 4ffebe8
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/6865eef82010220008a5dd3f
😎 Deploy Preview https://deploy-preview-51490--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 project configuration.

Copy link
Member

@dipesh-rawat dipesh-rawat left a comment

Choose a reason for hiding this comment

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

@sreeram-venkitesh Perhaps we should consider setting the dev-1.34 branch as the target branch rather than the main branch for this pull request. Since the change is related to the feature targeting stable in v1.34

@sreeram-venkitesh
Copy link
Member Author

Ah yes, I made this mistake last time as well. I'll clean this up this week.

@sreeram-venkitesh sreeram-venkitesh changed the base branch from main to dev-1.34 July 29, 2025 14:39
@sreeram-venkitesh
Copy link
Member Author

@dipesh-rawat I've fixed the base branch. Sorry for the delay on this. I believe this PR is ready for review. I don't see any other changes. The info on feature gates in the references for different components would be auto updated right? I don't remember doing anything for this last cycle when the gate graduated to beta. Thanks in advance!

CC: @michellengnx @rashansmith (Docs lead and assigned shadow)

@dipesh-rawat
Copy link
Member

@sreeram-venkitesh If this is ready for review, it might be a good time to remove the [WIP] from the title to help signal that to reviewers and approvers.

@sreeram-venkitesh sreeram-venkitesh changed the title [WIP] PodLifecycleSleepActionAllowZero to GA PodLifecycleSleepActionAllowZero to GA Jul 30, 2025
@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 Jul 30, 2025
@sreeram-venkitesh
Copy link
Member Author

Thanks Dipesh! Fixed now. I donno how I missed that 😂

@michellengnx
Copy link
Contributor

Hello @sreeram-venkitesh 👋! I'm reaching out from the Docs team. Just checking in as we approach Docs Freeze on Wednesday August 6, 2025 18:00 PDT. This documentation appears to still be under review. To meet the Docs Freeze, this PR must have a technical review as well as lgtm and approve labels applied, without any unaddressed comments or concerns from SIG Docs. Thank you!

@sreeram-venkitesh
Copy link
Member Author

@dipesh-rawat Can you please take a look at this PR. It should be a simple change from SIG Docs perspective. I will ping SIG Node folsk today for SIG review.

This is a beta-level feature default enabled by the `PodLifecycleSleepAction`
[feature gate](/docs/reference/command-line-tools-reference/feature-gates/).

{{< note >}}
Copy link
Member

Choose a reason for hiding this comment

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

From a docs perspective, the only thing to note is that this feature is mentioned only in this note block (aside from the feature gate description). With the note block being removed, there's currently no documentation page that references it.

Copy link
Contributor

Choose a reason for hiding this comment

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

A possible way forward would be to merge the content of the note in the Sleep bullet point above. Something like (draft written on the fly)

Since Kubernetes vX.Y it is possible to set a sleep duration of zero seconds (effectively a no-op) for your Sleep lifecycle hooks. You can disable this option by setting the beta level `PodLifecycleSleepActionAllowZero` feature gate to false.

@reylejano
Copy link
Member

/milestone 1.34
/approve

@k8s-ci-robot k8s-ci-robot added this to the 1.34 milestone Aug 6, 2025
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 6, 2025
@drewhagen
Copy link
Member

Hello @sreeram-venkitesh 👋!

I'm reaching out on behalf of the Release Docs team.

It looks like this PR has an approval from SIG Docs. Has SIG Node reviewed yet?

A friendly reminder that Docs Freeze is approaching today, at 18:00 PT (less than 9 hours). This documentation appears to still be under review. To meet the Docs Freeze, this PR must complete a technical review with SIG Node as well as lgtm and approve labels applied, without any unaddressed comments or concerns from SIG Docs.

If you wish to include this enhancement in 1.34 but are concerned about meeting the deadline, please consider proactively filing an exception request. Thanks!

cc: @kubernetes/sig-node-leads FYSA

Copy link
Member

@SergeyKanzhelev SergeyKanzhelev left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

let's clean up this part and if we need more docs, this can be tracked separately

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

LGTM label has been added.

Git tree hash: e47c1b22db0c51ab05ba2fafdb0ac537d1c57421

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: reylejano, SergeyKanzhelev

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

@k8s-ci-robot k8s-ci-robot merged commit 36a7865 into kubernetes:dev-1.34 Aug 6, 2025
6 checks passed
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. 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. sig/node Categorizes an issue or PR as relevant to SIG Node. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Development

Successfully merging this pull request may close these issues.

8 participants