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
priority and fairness: add production readiness review #2049
Conversation
253181d
to
308e3a5
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.
A couple more comments.
/lgtm @lavalamp - for approval |
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.
Some more suggestions, otherwise looks good.
df38283
to
e17fe09
Compare
/lgtm @lavalamp - for approval |
ping @wojtek-t for another LGTM -- one minor change since your last LGTM: as per Mike's comments, amended the answer to the question on how to enable/disable this feature. |
also ping @lavalamp for approval |
Just needs a couple fixes around what happens if you misconfigure, then it's good to go. |
Signed-off-by: Adhityaa Chandrasekar <adtac@google.com>
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adtac, lavalamp 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 |
I've filled in most of the questions in the PRR to the best of my (limited) knowledge -- please take a look and let me know if there are corrections to be made. I don't know enough to answer some of the questions - I'd appreciate your input on some of these questions.
/sig api-machinery
/cc @lavalamp @wojtek-t @ahg-g @MikeSpreitzer