From 7ad362bb3c5afa3999dd4998ad32775b77923b45 Mon Sep 17 00:00:00 2001 From: Matt Liggett Date: Fri, 9 Sep 2016 15:56:54 -0700 Subject: [PATCH] clarify weekend responsibilities --- docs/devel/on-call-build-cop.md | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/docs/devel/on-call-build-cop.md b/docs/devel/on-call-build-cop.md index 9cbea294505..530d7230a96 100644 --- a/docs/devel/on-call-build-cop.md +++ b/docs/devel/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