-
Notifications
You must be signed in to change notification settings - Fork 108
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
[to #423] add some guideline for triage critical bugs #503
Conversation
Signed-off-by: Jian Zhang <zjsariel@gmail.com>
1c20c5b
to
d563550
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
in this defect, gRPC timeout is not set for certain requests, which causes the | ||
requests can not be terminated as expected when the processing time is too long. | ||
|
||
### Major Defects |
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.
Should we add some brief description for these three lower defects? Or this guideline is still WIP.
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.
sure. updated, PTAL
Signed-off-by: Jian Zhang <zjsariel@gmail.com>
LGTM |
/merge |
/run-all-tests |
What problem does this PR solve?
Issue Number: to #423
Problem Description: the contribution guide lack guidelines about trading bugs.
What is changed and how does it work?
This PR adds a guideline for triaging critical bugs with one example.
Code changes
Check List for Tests
This PR has been tested by at least one of the following methods:
Side effects
Related changes