Skip to content
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

Adding draft SLA to issue-management.md file #163

Merged
merged 15 commits into from
Jul 31, 2019
39 changes: 38 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,41 @@ OK on GitHub.
We highly encourage to use line breaks in markdown files at `80` characters
wide. There are tools that can do it for you effectively. Please submit proposal
to include your editor settings required to enable this behavior so the out of
the box settings for this repository will be consistent.
the box settings for this repository will be consistent.

## SLA

For Issues

* For original poster (OP), after opening an urgent issue, please allow at least
(proposed) 1 business day before following up with the triager or assignee to
give time to work the issue and respond. Feel free to provide additional info/data
or ask further questions at anytime. Use URGENT label to distinguish from other
thomashchan1 marked this conversation as resolved.
Show resolved Hide resolved
issues and provide reason behind the urgency.

* For OP, after opening a non-urgent issue, please allow at least (proposed) 3
business days before follow up with the triager or assignee to give time to work
the issue and respond. Feel free to provide additional info/data or ask further
questions at anytime.

* For assignee, if the associated PR is merged, please proceed to close the issue,
thomashchan1 marked this conversation as resolved.
Show resolved Hide resolved
if not done already. For issues without PR and after providing
answers/resolution/workarounds, please allow at least (proposed) 2 business days
thomashchan1 marked this conversation as resolved.
Show resolved Hide resolved
for OP to respond/acknowledge before closing an issue.

For PRs

* Please allow a minimum of (proposed) 3 business days review period for PRs before
they could be merged.

* Maximum response time from reviewers/contributors: proposing 3 business days.

* If there is an exception to the general SLA, please highlight reasons in comments.
Valid exceptions are urgent fixes, typo, bug fixes, doc improvements, backfilling tests,
adding CI PRs or implementing what have already been discussed/agreed, etc..

* If SLA is violated and with no reason/justification comments, please be prepared to
accept another PR which could modify or rollback prior changes.

* Proposing to have 2 approvals from different companies for interface/design impacting changes.