Skip to content

Conversation

rikatz
Copy link
Member

@rikatz rikatz commented Oct 6, 2025

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Adds Provisional GEPs back to Website navigation

Which issue(s) this PR fixes:

Does this PR introduce a user-facing change?:

NONE

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/documentation Categorizes issue or PR as related to documentation. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 6, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rikatz
Once this PR has been reviewed and has the lgtm label, please assign danwinship 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/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 6, 2025
@rikatz
Copy link
Member Author

rikatz commented Oct 6, 2025

/cc @kflynn

As you wish :)

@k8s-ci-robot k8s-ci-robot requested a review from kflynn October 6, 2025 12:25
@kflynn
Copy link
Contributor

kflynn commented Oct 6, 2025

/lgtm

Thanks @rikatz! I think this is really important as we start looking to have things start in Provisional in order to make it into Experimental.

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

robscott commented Oct 7, 2025

/hold

I was actually one of the people that argued for the removal of provisional from navigation. This was for a few reasons:

  1. We are accepting a large number of provisional GEPs, and many of them may be dead ends/go nowhere. Highlighting these despite their provisional nature could lead to more confusion and/or disappointment.
  2. The existing GEP navigation is already overwhelming, this will only get worse as we add an increasing number of provisional GEPs.

If we want to include provisional GEPs somewhere on the website, filled with lots of disclaimers about what it means for something to be provisional, I can be OK with that, I'd just rather have it somewhere less prominent and with more warnings about the nature of provisional proposals.

@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 Oct 7, 2025
@kflynn
Copy link
Contributor

kflynn commented Oct 7, 2025

Sorry @robscott, I remember seeing your comments on this now and spaced on it. 🤦‍♂️

Any suggestions on what you think would be reasonable for the presentation? If we’re having provisional be an important part of the release cycle, we need to have them visible somehow.

@kflynn
Copy link
Contributor

kflynn commented Oct 7, 2025

(Maybe we should drop the TOC in the sidebar, and instead have links to category pages that then list the individual GEPs. 🤔)

@rikatz
Copy link
Member Author

rikatz commented Oct 7, 2025

(Maybe we should drop the TOC in the sidebar, and instead have links to category pages that then list the individual GEPs. 🤔)

I can rework this PR to follow this approach instead, rendering the pages experimental, provisional, etc with the GEPs. Will take me some time tho :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. kind/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants