Skip to content

Conversation

SubhasmitaSw
Copy link
Contributor

@SubhasmitaSw SubhasmitaSw commented Sep 30, 2025

What type of PR is this:

/kind documentation

What this PR does / why we need it:

  • Added clarification in the Before Code Freeze & Code Freeze Templates section

Addresses part of the umbrella issue #2375 to improve the Enhancements Role Handbook

Which issue(s) this PR fixes:

Special notes for your reviewer:

@k8s-ci-robot k8s-ci-robot added kind/documentation Categorizes issue or PR as related to documentation. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. area/release-team Issues or PRs related to the release-team subproject sig/release Categorizes an issue or PR as relevant to SIG Release. labels Sep 30, 2025
@k8s-ci-robot k8s-ci-robot requested a review from jenshu September 30, 2025 06:23
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: SubhasmitaSw
Once this PR has been reviewed and has the lgtm label, please assign rayandas for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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 added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 30, 2025
… fine to mark as tracked for code freeze for the purposes of enhancements tracking
@SubhasmitaSw SubhasmitaSw force-pushed the kep-bug-fixes-clarification branch from da5c356 to 413764f Compare September 30, 2025 06:29
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 30, 2025
@sreeram-venkitesh
Copy link
Member

CC: @jenshu

**All** enhancements going into the release must have their pull requests in a merge-ready state with approved and lgtm labels applied. This includes tests, and docs PRs opened by the due date. Due to the Prow queue, this may mean that approved PRs merge shortly after the deadline.
> **Note**: KEPs that contain bug fixes as part of their implementation work (even if the primary work is fixing existing issues rather than adding new features) are perfectly acceptable to mark as `Tracked for code freeze` for the purposes of enhancements tracking. The key requirement is that all code changes are properly reviewed and approved, regardless of whether they introduce new functionality or fix existing issues.
Copy link
Contributor

Choose a reason for hiding this comment

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

Just to clarify, do you mean that KEPs that only contain bug fixes should still be tracked? I think this goes against the messaging from https://groups.google.com/g/kubernetes-sig-release/c/LTzDa7OyBSA/m/IrKW973HAQAJ which says that bug fixes don't need to be tracked

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I think changes that do not affect current behavior don’t require a KEP and we don't have to track it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/release-team Issues or PRs related to the release-team subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants