diff options
| author | Kubernetes Submit Queue <k8s-merge-robot@users.noreply.github.com> | 2016-09-13 15:59:17 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-09-13 15:59:17 -0700 |
| commit | 3727a63edd22eadff51ab145667abcaa9c92b4ba (patch) | |
| tree | b04f6770f63d1a15e3b9d9b507a3df64f6da657a | |
| parent | bdf35f8f96ed2d73f0a7a0d30d6c71501f99d3fd (diff) | |
| parent | 2e227a61d0fdfa9c8890759b0cdfbbc79fea27eb (diff) | |
Merge pull request #32423 from mml/buildcop
Automatic merge from submit-queue
clarify weekend responsibilities
| -rw-r--r-- | on-call-build-cop.md | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/on-call-build-cop.md b/on-call-build-cop.md index 9cbea294..530d7230 100644 --- a/on-call-build-cop.md +++ b/on-call-build-cop.md @@ -139,7 +139,7 @@ or [flaky test builds](https://goto.google.com/k8s-test/view/Flaky/) are not your responsibility to monitor. The `Test owner:` in the job description will be automatically emailed if the job is failing. -* If you are a weekday oncall, ensure that PRs confirming to the following +* If you are oncall, ensure that PRs confirming to the following pre-requisites are being merged at a reasonable rate: * [Have been LGTMd](https://github.com/kubernetes/kubernetes/labels/lgtm) @@ -147,10 +147,9 @@ pre-requisites are being merged at a reasonable rate: * Author has signed CLA if applicable. -* If you are a weekend oncall, [never merge PRs manually](collab.md), instead -add the label "lgtm" to the PRs once they have been LGTMd and passed Travis; -this will cause merge-bot to merge them automatically (or make them easy to find -by the next oncall, who will merge them). +* Although the shift schedule shows you as being scheduled Monday to Monday, + working on the weekend is neither expected nor encouraged. Enjoy your time + off. * When the build is broken, roll back the PRs responsible ASAP |