-
Notifications
You must be signed in to change notification settings - Fork 238
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
Refresh documentation for waitForPodsReady #2541
Refresh documentation for waitForPodsReady #2541
Conversation
- Fix / improve field comments for the API - Rename the user-facing name of the feature to "all-or-nothing with ready Pods"
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mimowo 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 |
✅ Deploy Preview for kubernetes-sigs-kueue canceled.
|
// and defaults to true otherwise. | ||
// BlockAdmission when true, cluster queue will block admissions for all | ||
// subsequent jobs until the jobs reach the PodsReady=true condition. | ||
// It defaults to false if Enable is false and defaults to true otherwise. |
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.
uhm...
Maybe we can say that this setting is only honored if Enable
is set to true, instead of playing words about what the default is.
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.
Adjusted. I hope we could also make the defaulting logic simpler in the future. maybe just set it to false regardless or Enable?
Co-authored-by: Patryk Bundyra <73306396+PBundyra@users.noreply.github.com>
Co-authored-by: Patryk Bundyra <73306396+PBundyra@users.noreply.github.com>
Co-authored-by: Patryk Bundyra <73306396+PBundyra@users.noreply.github.com>
Co-authored-by: Patryk Bundyra <73306396+PBundyra@users.noreply.github.com>
Co-authored-by: Patryk Bundyra <73306396+PBundyra@users.noreply.github.com>
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.
Great documentation improvements!
Thank you!
LGTM
/lgtm |
LGTM label has been added. Git tree hash: 333c50da5a80c445e0171b9faf9d040158f4b7b2
|
/cherry-pick website |
@tenzen-y: new pull request created: #2554 In response to this:
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. |
What type of PR is this?
/kind documentation
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Other names of the feature / admin task considered:
Timeout-based all-or-nothing
Workload startup time
Any of these works for me.
Does this PR introduce a user-facing change?