From 0f0e1a85f3db5eca4dfd5eb376a822d68c504570 Mon Sep 17 00:00:00 2001 From: Xiaoning Ding Date: Thu, 16 Jan 2020 08:03:21 -0800 Subject: [PATCH] Update code owner files (#382) * Update project documents * Update code owners. --- OWNERS | 63 +++---- OWNERS_ALIASES | 450 ---------------------------------------------- SECURITY_CONTACTS | 17 -- 3 files changed, 22 insertions(+), 508 deletions(-) delete mode 100644 OWNERS_ALIASES delete mode 100644 SECURITY_CONTACTS diff --git a/OWNERS b/OWNERS index f7dffb6a56e..d7d728c40f9 100644 --- a/OWNERS +++ b/OWNERS @@ -1,41 +1,22 @@ -# See the OWNERS docs at https://go.k8s.io/owners - -filters: - ".*": - reviewers: - - brendandburns - - dchen1107 - - jbeda - - lavalamp - - smarterclayton - - thockin - - liggitt - approvers: - - bgrant0607 - - brendandburns - - dchen1107 - - jbeda - - monopole # To move code per kubernetes/community#598 - - lavalamp - - smarterclayton - - thockin - - wojtek-t - - liggitt - - # Bazel build infrastructure changes often touch files throughout the tree - "\\.bzl$": - reviewers: - - ixdy - approvers: - - ixdy - "BUILD(\\.bazel)?$": - approvers: - - ixdy - - # go.{mod,sum} files relate to go dependencies, and should be reviewed by the - # dep-approvers - "go\\.(mod|sum)$": - required_reviewers: - - kubernetes/dep-approvers - labels: - - area/dependency + +# This file lists the code owners of different areas of the project. +# These owners are the default code reviewers and approvers for PRs of the corresponding area. + + +Overall: + @panda98004, @XiaoningDing + +Controllers: + @dingyin, @Sindica, @vinaykul, @jshaofuturewei + +Node agent: + @XiaoningDing, @h-w-chen, @yb01, @pdgetrf, @vinaykul + +Scalability: + @haibinxie, @Sindica, @jeffzhu503, @sonyafenge + +Multi-tenancy: + @chenqianfzh, @XiaoningDing + +Documents: + @panda98004, @deepak-vij, @XiaoningDing, @zmn223 \ No newline at end of file diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES deleted file mode 100644 index 9a67c5ea4f5..00000000000 --- a/OWNERS_ALIASES +++ /dev/null @@ -1,450 +0,0 @@ -aliases: - # sig-auth subproject aliases - sig-auth-audit-approvers: - - sttts - - tallclair - sig-auth-audit-reviewers: - - CaoShuFeng - - hzxuzhonghu - - lavalamp - - sttts - - tallclair - - sig-auth-authenticators-approvers: - - deads2k - - liggitt - - mikedanese - sig-auth-authenticators-reviewers: - - deads2k - - enj - - jianhuiz - - lavalamp - - liggitt - - mbohlool - - mikedanese - - sttts - - wojtek-t - - sig-auth-authorizers-approvers: - - deads2k - - liggitt - - mikedanese - sig-auth-authorizers-reviewers: - - david-mcmahon - - deads2k - - dims - - enj - - erictune - - jianhuiz - - krousey - - lavalamp - - liggitt - - mbohlool - - mikedanese - - mml - - ncdc - - nikhiljindal - - smarterclayton - - sttts - - thockin - - wojtek-t - - sig-auth-certificates-approvers: - - liggitt - - mikedanese - - smarterclayton - sig-auth-certificates-reviewers: - - awly - - caesarxuchao - - david-mcmahon - - deads2k - - dims - - enj - - errordeveloper - - hongchaodeng - - jianhuiz - - lavalamp - - liggitt - - mbohlool - - mikedanese - - smarterclayton - - sttts - - thockin - - timothysc - - wojtek-t - - sig-auth-encryption-at-rest-approvers: - - immutableT - - smarterclayton - sig-auth-encryption-at-rest-reviewers: - - enj - - immutableT - - lavalamp - - liggitt - - sakshamsharma - - smarterclayton - - wojtek-t - - sig-auth-node-isolation-approvers: - - deads2k - - liggitt - - mikedanese - - tallclair - sig-auth-node-isolation-reviewers: - - deads2k - - liggitt - - mikedanese - - tallclair - - sig-auth-policy-approvers: - - deads2k - - liggitt - - tallclair - sig-auth-policy-reviewers: - - deads2k - - hongchaodeng - - jianhuiz - - liggitt - - mbohlool - - pweil- - - tallclair - - krmayankk - - sig-auth-serviceaccounts-approvers: - - deads2k - - liggitt - - mikedanese - sig-auth-serviceaccounts-reviewers: - - awly - - deads2k - - enj - - liggitt - - mikedanese - - WanLinghao - - sig-storage-reviewers: - - saad-ali - - childsb - - sig-scheduling-maintainers: - - bsalamat - - k82cn - - wojtek-t - - ravisantoshgudimetla - - Huang-Wei - sig-scheduling: - - bsalamat - - k82cn - - resouer - - ravisantoshgudimetla - - misterikkit - - Huang-Wei - - wgliang - - sig-cli-maintainers: - - adohe - - brendandburns - - deads2k - - janetkuo - - liggitt - - seans3 - - monopole - - droot - - apelisse - - mengqiy - - smarterclayton - - soltysh - sig-cli: - - adohe - - deads2k - - derekwaynecarr - - dixudx - - dims - - dshulyak - - eparis - - ghodss - - juanvallejo - - mengqiy - - rootfs - - seans3 - - shiywang - - smarterclayton - - soltysh - sig-testing-reviewers: - - bentheelder - - cblecker - - fejta - - krzyzacy - - ixdy - - spiffxp - sig-testing-approvers: - - bentheelder - - cblecker - - fejta - - krzyzacy - - ixdy - - spiffxp - sig-node-reviewers: - - Random-Liu - - dashpole - - dchen1107 - - derekwaynecarr - - dims - - feiskyer - - mtaufen - - pmorie - - resouer - - sjenning - - sjpotter - - tallclair - - tmrts - - vishh - - yifan-gu - - yujuhong - - krmayankk - sig-network-approvers: - - bowei - - caseydavenport - - danwinship - - dcbw - - dnardo - - freehan - - johnbelamaric - - mrhohn - - nicksardo - - thockin - sig-network-reviewers: - - bowei - - caseydavenport - - danwinship - - dcbw - - dnardo - - freehan - - johnbelamaric - - mrhohn - - nicksardo - - thockin - - rramkumar1 - - cmluciano - - m1093782566 - sig-apps-approvers: - - kow3ns - - janetkuo - - soltysh - - tnozicka - - erictune - - smarterclayton - sig-apps-reviewers: - - enisoc - - erictune - - foxish - - janetkuo - - kow3ns - - mortent - - smarterclayton - - soltysh - - tnozicka - - krmayankk - sig-autoscaling-maintainers: - - aleksandra-malinowska - - bskiba - - MaciekPytel - - mwielgus - - api-approvers: - - erictune - - lavalamp - - smarterclayton - - thockin - - liggitt - # - bgrant0607 # manual escalations only - - # subsets of api-approvers by sig area to help focus approval requests to those with domain knowledge - sig-api-machinery-api-approvers: - - lavalamp - - liggitt - - smarterclayton - - sig-apps-api-approvers: - - erictune - - lavalamp - - liggitt - - smarterclayton - - sig-auth-api-approvers: - - liggitt - - smarterclayton - - sig-cli-api-approvers: - - liggitt - - smarterclayton - - sig-cloud-provider-api-approvers: - - liggitt - - thockin - - sig-cluster-lifecycle-api-approvers: - - liggitt - - smarterclayton - - sig-network-api-approvers: - - smarterclayton - - thockin - - sig-network-api-reviewers: - - caseydavenport - - cmluciano - - danwinship - - thockin - - sig-node-api-approvers: - - smarterclayton - - thockin - - sig-scheduling-api-approvers: - - lavalamp - - smarterclayton - - thockin - - sig-storage-api-approvers: - - liggitt - - thockin - - sig-windows-api-approvers: - - smarterclayton - - thockin - - liggitt - - api-reviewers: - - erictune - - lavalamp - - smarterclayton - - thockin - - liggitt - - wojtek-t - - deads2k - - yujuhong - - brendandburns - - derekwaynecarr - - caesarxuchao - - vishh - - mikedanese - - nikhiljindal - - gmarek - - davidopp - - pmorie - - sttts - - dchen1107 - - saad-ali - - zmerlynn - - luxas - - janetkuo - - justinsb - - pwittrock - - roberthbailey - - ncdc - - tallclair - - yifan-gu - - eparis - - mwielgus - - timothysc - - soltysh - - piosz - - jsafrane - - jbeda - - # api-reviewers targeted by sig area - # see https://git.k8s.io/community/sig-architecture/api-review-process.md#training-reviews - - sig-api-machinery-api-reviewers: - - caesarxuchao - - deads2k - - jpbetz - - sttts - - # sig-apps-api-reviewers: - # - - # - - - sig-auth-api-reviewers: - - enj - - mikedanese - - sig-cli-api-reviewers: - - pwittrock - - soltysh - - sig-cloud-provider-api-reviewers: - - andrewsykim - - cheftako - - # sig-cluster-lifecycle-api-reviewers: - # - - # - - - sig-node-api-reviewers: - - dchen1107 - - derekwaynecarr - - tallclair - - yujuhong - - sig-scheduling-api-reviewers: - - bsalamat - - k82cn - - sig-storage-api-reviewers: - - saad-ali - - msau42 - - jsafrane - - - sig-windows-api-reviewers: - - patricklang - - ddebroy - - benmoss - - dep-approvers: - - apelisse - - BenTheElder - - cblecker - - dims - - thockin - - sttts - - soltysh - feature-approvers: - - bgrant0607 # Architecture - - brancz # Instrumentation - - bsalamat # Scheduling - - calebamiles # Release - - caseydavenport # Network - - childsb # Storage - - countspongebob # Scalability - - csbell # Multicluster - - dcbw # Network - - dchen1107 # Node - - deads2k # API Machinery - - derekwaynecarr # Node - - dghubble # On Premise - - jdumars # Architecture, Cluster Ops, Release - - kow3ns # Apps - - lavalamp # API Machinery - - liggitt # Auth - - lukemarsden # Cluster Lifecycle - - luxas # Cluster Lifecycle - - marcoceppi # On Premise - - mattfarina # Apps - - michmike # Windows - - mwielgus # Autoscaling - - piosz # Instrumentation - - prydonius # Apps - - pwittrock # CLI - - quinton-hoole # Multicluster - - roberthbailey # Cluster Lifecycle - - saad-ali # Storage - - seans3 # CLI - - soltysh # CLI - - tallclair # Auth - - thockin # Network - - timothysc # Cluster Lifecycle, Scheduling - - wojtek-t # Scalability - - zehicle # Cluster Ops diff --git a/SECURITY_CONTACTS b/SECURITY_CONTACTS deleted file mode 100644 index 70b7cf9a657..00000000000 --- a/SECURITY_CONTACTS +++ /dev/null @@ -1,17 +0,0 @@ -# Defined below are the security contacts for this repo. -# -# They are the contact point for the Product Security Committee to reach out -# to for triaging and handling of incoming issues. -# -# The below names agree to abide by the -# [Embargo Policy](https://git.k8s.io/security/private-distributors-list.md#embargo-policy) -# and will be removed and replaced if they violate that agreement. -# -# DO NOT REPORT SECURITY VULNERABILITIES DIRECTLY TO THESE NAMES, FOLLOW THE -# INSTRUCTIONS AT https://kubernetes.io/security/ - -cjcullen -jessfraz -liggitt -philips -tallclair