Merge pull request #42206 from sftim/20230725_update_owners

k8s-ci-robot · web-flow · commit b1479aa8ea16 · 2024-04-18T09:19:17.000-07:00
Switch to finer controls on approver access
diff --git a/.github/OWNERS b/.github/OWNERS
@@ -1,7 +1,7 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
 reviewers:
-- sig-docs-en-reviews # Defined in OWNERS_ALIASES
+- sig-docs-website-owners # Defined in OWNERS_ALIASES
 
 approvers:
-- sig-docs-en-owners # Defined in OWNERS_ALIASES
+- sig-docs-website-owners # Defined in OWNERS_ALIASES
diff --git a/OWNERS b/OWNERS
@@ -1,10 +1,10 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
 reviewers:
-- sig-docs-en-reviews # Defined in OWNERS_ALIASES
+- sig-docs-website-owners # Defined in OWNERS_ALIASES
 
 approvers:
-- sig-docs-en-owners # Defined in OWNERS_ALIASES
+- sig-docs-website-owners # Defined in OWNERS_ALIASES
 
 emeritus_approvers:
 # - chenopis, commented out to disable PR assignments
diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES
@@ -8,10 +8,28 @@ aliases:
     - mrbobbytables
     - nate-double-u
     - sftim
+  sig-docs-website-owners: # Admins for overall website
+    - divya-mohan0209
+    - natalisucks
+    - reylejano
+    - sftim
+    - tengqm
+    - drewhagen # RT 1.30 Docs Lead
+    - katcosgrove # RT 1.30 Lead
   sig-docs-localization-owners: # Admins for localization content
     - a-mccarthy
     - divya-mohan0209
     - natalisucks
+    - nate-double-u
+    - reylejano
+    - sftim
+    - seokho-son
+    - tengqm
+  sig-docs-localization-reviewers: # PR reviews for localization changes
+    - a-mccarthy
+    - divya-mohan0209
+    - natalisucks
+    - nate-double-u
     - reylejano
     - sftim
     - seokho-son
@@ -26,8 +44,6 @@ aliases:
     - celestehorgan
     - dipesh-rawat
     - divya-mohan0209
-    - drewhagen # RT 1.30 Docs Lead
-    - katcosgrove # RT 1.30 Lead
     - natalisucks
     - nate-double-u
     - reylejano
diff --git a/assets/OWNERS b/assets/OWNERS
@@ -0,0 +1,9 @@
+# Label certain changes as web development
+
+filters:
+  "\\.js":
+    labels:
+    - area/web-development
+  "\\.scss$":
+    labels:
+    - area/web-development
diff --git a/assets/images/OWNERS b/assets/images/OWNERS
@@ -0,0 +1,22 @@
+# See the OWNERS docs at https://go.k8s.io/owners
+
+# This is the directory for common images. Allow English localization
+# reviewers to review and approve.
+# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
+
+reviewers:
+- sig-docs-en-reviews
+
+approvers:
+- sig-docs-en-owners
+
+filters:
+  "\\.svg":
+    labels:
+    - area/web-development
+  "\\.png$":
+    labels:
+    - area/web-development
+  "\\.jpe?g$":
+    labels:
+    - area/web-development
diff --git a/content/OWNERS b/content/OWNERS
@@ -0,0 +1,11 @@
+# See the OWNERS docs at https://go.k8s.io/owners
+
+# This is the directory for all source content.
+# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
+
+reviewers:
+- sig-docs-localization-reviewers
+
+approvers:
+- sig-docs-localization-owners
+- sig-docs-website-owners
diff --git a/content/de/OWNERS b/content/de/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-de-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/de
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/es/OWNERS b/content/es/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-es-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/es
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/fr/OWNERS b/content/fr/OWNERS
@@ -8,6 +8,15 @@ reviewers:
 
 approvers:
 - sig-docs-fr-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/fr
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
+
diff --git a/content/hi/OWNERS b/content/hi/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-hi-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/hi
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/id/OWNERS b/content/id/OWNERS
@@ -1,13 +1,21 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
-# This is the localization project for Bahasa.
+# This is the localization project for Bahasa Indonesia.
 # Teams and members are visible at https://github.com/orgs/kubernetes/teams.
 
 reviewers:
 - sig-docs-id-reviews
 
 approvers:
 - sig-docs-id-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/id
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/it/OWNERS b/content/it/OWNERS
@@ -6,6 +6,14 @@ reviewers:
 
 approvers:
 - sig-docs-it-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
-- language/it
+- area/localization
+- language/it
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/ja/OWNERS b/content/ja/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-ja-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/ja
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/ko/OWNERS b/content/ko/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-ko-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/ko
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/pl/OWNERS b/content/pl/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-pl-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/pl
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/pt-br/OWNERS b/content/pt-br/OWNERS
@@ -1,11 +1,19 @@
-# This is the localization project for Portuguese.
+# This is the localization project for (Brazilian) Portuguese.
 # Teams and members are visible at https://github.com/orgs/kubernetes/teams.
 
 reviewers:
 - sig-docs-pt-reviews
 
 approvers:
 - sig-docs-pt-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/pt
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/ru/OWNERS b/content/ru/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-ru-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/ru
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/uk/OWNERS b/content/uk/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-uk-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/uk
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/vi/OWNERS b/content/vi/OWNERS
@@ -8,6 +8,14 @@ reviewers:
 
 approvers:
 - sig-docs-vi-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/vi
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/content/zh-cn/OWNERS b/content/zh-cn/OWNERS
@@ -1,13 +1,21 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
-# This is the localization project for Chinese.
+# This is the localization project for Chinese (simplified / People's Republic of China)
 # Teams and members are visible at https://github.com/orgs/kubernetes/teams.
 
 reviewers:
 - sig-docs-zh-reviews
 
 approvers:
 - sig-docs-zh-owners
+- sig-docs-website-owners # important due to no_parent_owners option
 
 labels:
+- area/localization
 - language/zh
+
+# Disable inheritance so that the localization team don't end up as approvers for all
+# languages.
+# https://github.com/kubernetes/community/blob/master/communication/website-guidelines.md
+options:
+  no_parent_owners: true
diff --git a/data/canonical-tags/OWNERS b/data/canonical-tags/OWNERS
@@ -0,0 +1,9 @@
+# See the OWNERS docs at https://go.k8s.io/owners
+
+# This is where we manage the categories for glossary entries.
+
+reviewers:
+- sig-docs-en-reviews
+
+approvers:
+- sig-docs-website-owners
diff --git a/data/i18n/OWNERS b/data/i18n/OWNERS
@@ -0,0 +1,11 @@
+# See the OWNERS docs at https://go.k8s.io/owners
+
+# This is the directory for all source content.
+# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
+
+reviewers:
+- sig-docs-localization-reviewers
+
+approvers:
+- sig-docs-localization-owners
+- sig-docs-website-owners
diff --git a/scripts/OWNERS b/scripts/OWNERS
@@ -0,0 +1,2 @@
+# Scripts to help manage the website
+# No special ownership (as yet)
diff --git a/scripts/releng/OWNERS b/scripts/releng/OWNERS
@@ -4,11 +4,11 @@
 # Teams and members are visible at https://github.com/orgs/kubernetes/teams.
 
 reviewers:
-  - sig-docs-en-reviews
+  - sig-docs-website-owners
   - release-engineering-reviewers
 
 approvers:
-  - sig-docs-en-owners
+  - sig-docs-website-owners
   - sig-release-leads
   - release-engineering-approvers
 
diff --git a/static/css/OWNERS b/static/css/OWNERS
@@ -0,0 +1,6 @@
+# Label certain changes as web development
+
+filters:
+  "\\.css$":
+    labels:
+    - area/web-development
diff --git a/static/js/OWNERS b/static/js/OWNERS
@@ -0,0 +1,9 @@
+# Label certain changes as web development
+
+approvers:
+- sig-docs-website-owners # Defined in OWNERS_ALIASES
+
+filters:
+  "\\.js":
+    labels:
+    - area/web-development
