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

[KEP-5073] Declarative Validation Of Kubernetes Native Types With validation-gen #5073

Open
6 tasks
aaron-prindle opened this issue Jan 22, 2025 · 12 comments
Open
6 tasks
Assignees
Labels
lead-opted-in Denotes that an issue has been opted in to a release sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. stage/beta Denotes an issue tracking an enhancement targeted for Beta status tracked/yes Denotes an enhancement issue is actively being tracked by the Release Team
Milestone

Comments

@aaron-prindle
Copy link
Contributor

aaron-prindle commented Jan 22, 2025

Enhancement Description

  • One-line enhancement description (can be used as a release note): Implement declarative validation of Kubernetes native types using validation-gen to generate validation code.
  • Kubernetes Enhancement Proposal: KEP 5073
  • Discussion Link: KEP-5073: Declarative Validation of Kubernetes Native Types With validation-gen #5074
  • Primary contact (assignee): @aprindle
  • Responsible SIGs: sig-api-machinery
  • Enhancement target (which target equals to which milestone):
    • Beta release target (x.y): 1.33
    • Stable release target (x.y):

Beta

Please keep this description up to date. This will help the Enhancement Team to track the evolution of the enhancement efficiently.

@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Jan 22, 2025
@aaron-prindle aaron-prindle changed the title [KEP-NNNN] Declarative Validation Of Kubernetes Native Types With validation-gen [KEP-5073] Declarative Validation Of Kubernetes Native Types With validation-gen Jan 22, 2025
@aaron-prindle
Copy link
Contributor Author

/sig api-machinery
/wg api-machinery

@k8s-ci-robot k8s-ci-robot added the sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. label Jan 22, 2025
@k8s-ci-robot
Copy link
Contributor

@aaron-prindle: The label(s) wg/api-machinery cannot be applied, because the repository doesn't have them.

In response to this:

/sig api-machinery
/wg api-machinery

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot removed the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Jan 22, 2025
@jpbetz
Copy link
Contributor

jpbetz commented Jan 22, 2025

/label lead-opted-in
/milestone v1.33

@k8s-ci-robot k8s-ci-robot added this to the v1.33 milestone Jan 22, 2025
@k8s-ci-robot k8s-ci-robot added the lead-opted-in Denotes that an issue has been opted in to a release label Jan 22, 2025
@thockin thockin self-assigned this Jan 28, 2025
@thockin thockin moved this to Pre-alpha in KEPs I am tracking Jan 29, 2025
@lzung
Copy link

lzung commented Feb 2, 2025

Hello @aaron-prindle 👋, v1.33 Enhancements team here.

Just checking in as we approach enhancements freeze on 02:00 UTC Friday 14th February 2025 / 19:00 PDT Thursday 13th February 2025.

This enhancement is targeting stage beta for v1.33 (correct me, if otherwise)
/stage beta

Here's where this enhancement currently stands:

  • KEP readme using the latest template has been merged into the k/enhancements repo.
  • KEP status is marked as implementable for latest-milestone: v1.33.
  • KEP readme has up-to-date graduation criteria
  • KEP has a production readiness review that has been completed and merged into k/enhancements. (For more information on the PRR process, check here). If your production readiness review is not completed yet, please make sure to fill the production readiness questionnaire in your KEP by the PRR Freeze deadline on Thursday 6th February 2025 so that the PRR team has enough time to review your KEP.

For this KEP, we would just need to update the following:

  • Create the KEP readme using the latest template and merge it in the k/enhancements repo.
  • Ensure that the KEP has undergone a production readiness review and has been merged into k/enhancements.

The status of this enhancement is marked as At risk for enhancements freeze. Please keep the issue description up-to-date with appropriate stages as well.

If you anticipate missing enhancements freeze, you can file an exception request in advance. Thank you!

@k8s-ci-robot k8s-ci-robot added the stage/alpha Denotes an issue tracking an enhancement targeted for Alpha status label Feb 2, 2025
@lzung lzung moved this to At risk for enhancements freeze in 1.33 Enhancements Tracking Feb 2, 2025
@lzung lzung added stage/beta Denotes an issue tracking an enhancement targeted for Beta status and removed stage/alpha Denotes an issue tracking an enhancement targeted for Alpha status labels Feb 2, 2025
@dipesh-rawat
Copy link
Member

Hi @aaron-prindle 👋, 1.33 Enhancements team here,

Just a quick friendly reminder as we approach the enhancements freeze later this week, at 02:00 UTC Friday 14th February 2025 / 19:00 PDT Thursday 13th February 2025.

The current status of this enhancement is marked as At risk for enhancement freeze. There are a few requirements mentioned in the comment #5073 (comment) that still need to be completed.

If you anticipate missing enhancements freeze, you can file an exception request in advance. Thank you!

@aaron-prindle
Copy link
Contributor Author

aaron-prindle commented Feb 10, 2025

Thanks @lzung and @dipesh-rawat. The goal for the KEP currently is to get it merged before the enhancements freeze for 1.33. From the comments above I believe for this KEP, we would just need to update/complete the following:

We are currently in the process (see KEP PR here - #5074) and are working with the necessary reviewers to get this in before the enhancements freeze (assuming no major blockers from reviewers). Currently I believe we will be able to get the KEP merged prior the enhancements freeze date.

@hacktivist123
Copy link

Hello @aaron-prindle 👋, v1.33 Docs Shadow here.

Does this enhancement work planned for v1.33 require any new docs or modification to existing docs?
If so, please follow the steps here to open a PR against dev-1.33 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thursday 27th February 2025 18:00 PDT.

Also, take a look at Documenting for a release to get yourself familiarize with the docs requirement for the release.
Thank you!

@aaron-prindle
Copy link
Contributor Author

aaron-prindle commented Feb 12, 2025

@hacktivist123 thanks for the information here! The KEP adds 2 feature gates which should be documented in the k/website repo. I have followed the steps posted (link) and submitted a draft PR to k/website, PR here:
kubernetes/website#49732

@dipesh-rawat
Copy link
Member

Hello @aaron-prindle 👋, v1.33 Enhancements team here.

Unfortunately, this enhancement did not meet requirements for enhancements freeze.

If you still wish to progress this enhancement in v1.33, please file an exception request as soon as possible, within three days. If you have any questions, you can reach out in the #release-enhancements channel on Slack and we'll be happy to help. Thanks!

Looks like PR #5074 to update the KEP files misses to add the Release Signoff Checklist in the README file from the latest template (here).

/milestone clear

@k8s-ci-robot k8s-ci-robot removed this from the v1.33 milestone Feb 14, 2025
@dipesh-rawat dipesh-rawat moved this from At risk for enhancements freeze to Removed from Milestone in 1.33 Enhancements Tracking Feb 14, 2025
@aaron-prindle
Copy link
Contributor Author

@dipesh-rawat - thanks for the information here. I have submitted a PR to fix the issue with the Release Signoff Checklist:
#5171

The KEP itself was reviewed and approved by a Sig Lead - @deads2k
and had PRR review and approval from - @jpbetz

I have submitted an exception request following the guide here to:
[email protected],
[email protected],
[email protected]
If possible, can you confirm that with that PR the KEP would be compliant and be allowed into 1.33? I am not sure how that section was missed (sorry!) so just want to maybe get some confirmation that we would meet the necessary criteria if that PR is merged. Thanks

@dipesh-rawat
Copy link
Member

@aaron-prindle Since the release team has APPROVED the exception request here. This will be considered to be added to the milestone for v1.33 release.

@dipesh-rawat
Copy link
Member

@aaron-prindle Now that PRs #5074 #5171 has been merged, all the KEP requirements are in place and merged into k/enhancements, this enhancement is all good for the upcoming enhancements freeze. 🚀

The status of this enhancement is marked as tracked for enhancement freeze. Please keep the issue description up-to-date with appropriate stages as well. Thank you!
(cc: @lzung)

/milestone v1.33
/label tracked/yes

@k8s-ci-robot k8s-ci-robot added the tracked/yes Denotes an enhancement issue is actively being tracked by the Release Team label Feb 14, 2025
@k8s-ci-robot k8s-ci-robot added this to the v1.33 milestone Feb 14, 2025
@dipesh-rawat dipesh-rawat moved this from Removed from Milestone to Tracked for enhancements freeze in 1.33 Enhancements Tracking Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lead-opted-in Denotes that an issue has been opted in to a release sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. stage/beta Denotes an issue tracking an enhancement targeted for Beta status tracked/yes Denotes an enhancement issue is actively being tracked by the Release Team
Projects
Status: Tracked for enhancements freeze
Development

No branches or pull requests

7 participants