diff options
| -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 |