From 69fce507114f541f33e2117e06fcad8a9b148d15 Mon Sep 17 00:00:00 2001 From: "Dr. Stefan Schimanski" Date: Mon, 25 Jan 2021 13:43:18 +0100 Subject: [PATCH] UPSTREAM: : Add DOWNSTREAM_OWNERS UPSTREAM: : clarify downstream approver rules --- DOWNSTREAM_OWNERS | 32 ++++++++++++++++++++++++++++++++ pkg/kubelet/DOWNSTREAM_OWNERS | 17 +++++++++++++++++ 2 files changed, 49 insertions(+) create mode 100644 DOWNSTREAM_OWNERS create mode 100644 pkg/kubelet/DOWNSTREAM_OWNERS diff --git a/DOWNSTREAM_OWNERS b/DOWNSTREAM_OWNERS new file mode 100644 index 0000000000000..cfed18eafb646 --- /dev/null +++ b/DOWNSTREAM_OWNERS @@ -0,0 +1,32 @@ +# See the OWNERS docs at https://go.k8s.io/owners + +filters: + ".*": + # Downstream reviewers, don't have to match those in OWNERS + reviewers: + - deads2k + - sttts + - soltysh + - mfojtik + - marun + - tnozicka + + # Approvers are limited to the team that manages rebases and pays the price for carries that are introduced + approvers: + - deads2k + - sttts + - soltysh + - mfojtik + - marun + - tnozicka + + "^\\.go.(mod|sum)$": + labels: + - "vendor-update" + "^vendor/.*": + labels: + - "vendor-update" + "^staging/.*": + labels: + - "vendor-update" +component: kube-apiserver diff --git a/pkg/kubelet/DOWNSTREAM_OWNERS b/pkg/kubelet/DOWNSTREAM_OWNERS new file mode 100644 index 0000000000000..d484fa4fc246a --- /dev/null +++ b/pkg/kubelet/DOWNSTREAM_OWNERS @@ -0,0 +1,17 @@ +# See the OWNERS docs at https://go.k8s.io/owners + +# Downstream reviewers, don't have to match those in OWNERS +reviewers: + - rphillips + - sjenning + - mrunalp + +# Sub-package approvers from upstream with permission to approve downstream backports following these rules: +# - they MUST be approvers upstream (here compare https://github.com/kubernetes/kubernetes/blob/17bb2fc050ec786b60db7d8d6d4d3ac8eeac205b/pkg/kubelet/OWNERS#L10-L11) +# - they may approve "UPSTREAM: : ..." changes that merged upstream. +# - carry patches for "UPSTREAM: : ..." and any unmerged PRs of the previous kind will have to be approved by the top-level approvers. +approvers: + - sjenning + - mrunalp + +component: node