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
Document dynamic cardinality enforcement (KEP 2305) #41998
Conversation
d7963f7
to
02378c3
Compare
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Linked to kubernetes/enhancements#2305 |
Hello @rexagod 👋 please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 25th July 2023. Thank you! |
Hello, @katcosgrove. The linked k/k PR is no longer tracked and has been moved out of the milestone. I’m assuming this will have to be deferred till the next release? |
Ah okay! Updating our tracking sheet, thank you! |
Changed base to target latest. |
Hello @rexagod 👋, I cannot see any file changes on this PR. Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 14th November 2023. Thank you! |
Hello @rexagod @logicalhan 👋, gentle reminder to take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 14th November 2023. Thank you! |
02378c3
to
d8b00cf
Compare
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
/lgtm cancel |
32ea39f
to
5952d91
Compare
@sftim Do I need Han's LGTM here again for this to be merged (after all reviews are addressed)? US folks are out due to Thanksgiving, so that might not be possible within the limited time-frame. |
5952d91
to
9ef98a4
Compare
Hi @rexagod! I'm the Docs lead for v1.29. You will need an LGTM from someone, yes. If @logicalhan is still out after Thanksgiving, someone from your SIG's leadership will suffice. Would you like me to find someone for you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks but we really do need these docs revised in line with the reviews so far. /hold |
Signed-off-by: Pranshu Srivastava <rexagod@gmail.com>
9ef98a4
to
90c282e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
LGTM label has been added. Git tree hash: 75be3702ee6815340fa272c357bf7fbef5ee21ae
|
/hold cancel |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: logicalhan, 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 |
document kep-2305: metric cardinality enforcement