From 28315f8525f691815ed79445a19ff4a31fa1a2c4 Mon Sep 17 00:00:00 2001
From: Andrew Ballantyne <aballant@redhat.com>
Date: Tue, 29 Aug 2023 15:02:15 -0400
Subject: [PATCH] Add approvers & include SME docs

---
 .github/pull_request_template.md |  2 +-
 OWNERS                           | 15 ++++++--
 docs/smes.md                     | 65 ++++++++++++++++++++++++++++++++
 3 files changed, 78 insertions(+), 4 deletions(-)
 create mode 100644 docs/smes.md

diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
index 1e752b78d8..8f470dc456 100644
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -28,7 +28,7 @@ Self checklist (all need to be checked):
 If you have UI changes: 
 <!--- You can ignore these if you are doing manifest, backend, internal logic, etc changes; aka non-UI / visual changes -->
 - [ ] Included any necessary screenshots or gifs if it was a UI change.
-- [ ] Included tags to the UX team if it was a UI/UX change.
+- [ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the [SMEs](https://github.com/opendatahub-io/odh-dashboard/tree/main/docs/smes.md) section).
 
 After the PR is posted & before it merges:
 - [ ] The developer has tested their solution on a cluster by using the image produced by the PR to `main`
diff --git a/OWNERS b/OWNERS
index 6cf86d8d1c..bf019d3d5f 100644
--- a/OWNERS
+++ b/OWNERS
@@ -1,8 +1,17 @@
 approvers:
 - andrewballantyne
+- lucferbux
+- alexcreasy
+- christianvogt
 
 reviewers:
-- DaoDaoNoCode
-- lucferbux
-- Gkrumbach07
 - alexcreasy
+- christianvogt
+- uidoyen
+- Gkrumbach07
+- lucferbux
+- DaoDaoNoCode
+- manaswinidas
+- pnaik1
+- ppadti
+- dpanshug
diff --git a/docs/smes.md b/docs/smes.md
new file mode 100644
index 0000000000..614dc6d8d3
--- /dev/null
+++ b/docs/smes.md
@@ -0,0 +1,65 @@
+# Subject Matter Experts (SMEs)
+
+A given subject matter expert is not necessarily the most knowledgeable in the area, but they are the one who probably knows the most about it when it was originally done or has a responsibility to expand their knowledge to know about the area going forward. Contacting them first will help with delegation of responsibilities at the Dashboard level.
+
+This will detail out former (or current) feature leads, area leads (has a responsibility to understand the area), as well as any other notable position in relation to the area. If you need to talk to someone or ping someone for a review, this information should help you determine who.
+
+Below there will be some terms like “previous” and “backup”, these are for additional context. The way you can read each are as follows:
+- **previous** – the initial SME in the area. If you need legacy context, this person may be able to help
+- **backup** – a good person to lean on if there is a need for any 2nd opinions, for bouncing ideas off of, or any larger discussion about direction
+- **and** – Ping both during conversations – could be onboarding, could be a need to share information, best get both people involved at the same time
+
+General Dashboard ownership
+- Infrastructure / direction
+    - Architect: `Andrew` ([andrewballantyne])
+    - General UX: `Kyle` ([kywalker-rh])
+    - App Text: `Katie` ([kaedward])
+- Testing (Integration, Unit, etc)
+    - Area lead: `Gage` ([Gkrumbach07])
+- Performance
+    - Area lead: `Lucas` ([lucferbux])
+
+Dashboard features
+- Data Science Projects
+    - Feature lead: `Andrew` ([andrewballantyne])
+    - UX: `Kyle` ([kywalker-rh]) **and** `Kun` ([xianli123])
+- Data Science Pipelines
+    - Feature lead: `Andrew` ([andrewballantyne])
+    - UX: `Yan` ([yannnz])
+        - Previous: `Kyle` ([kywalker-rh])
+- Explainability, Bias
+    - Feature lead: `Alex` ([alexcreasy])
+    - UX: `Vince` ([vconzola])
+- Model Serving (Custom runtimes, general Model Serving)
+    - Feature lead: `Lucas` ([lucferbux])
+    - UX: `Vince` ([vconzola])
+- Model Serving - Performance metrics
+    - Feature lead: `Andrew` ([andrewballantyne])
+    - UX: `Vince` ([vconzola])
+- BYON - Custom Notebook Images
+    - Feature lead: `Juntao` ([DaoDaoNoCode])
+        - Backup: `Andrew` ([andrewballantyne])
+    - UX: `Vince` ([vconzola])
+- Habana / Accelerators
+    - Feature lead: `Gage` ([Gkrumbach07]);
+        - Backup: `Andrew` ([andrewballantyne])
+    - UX: `Yan` ([yannnz])
+- Model Registry
+    - Feature lead: TBD
+    - UX: `Sim` ([simrandhaliw])  **and** `Haley` ([yih-wang])
+
+<!-- Developers -->
+[andrewballantyne]: https://github.com/andrewballantyne
+[Gkrumbach07]: https://github.com/Gkrumbach07
+[lucferbux]: https://github.com/lucferbux
+[alexcreasy]: https://github.com/alexcreasy
+[DaoDaoNoCode]: https://github.com/DaoDaoNoCode
+
+<!-- UX -->
+[kywalker-rh]: https://github.com/kywalker-rh
+[kaedward]: https://github.com/kaedward
+[xianli123]: https://github.com/xianli123
+[vconzola]: https://github.com/vconzola
+[yannnz]: https://github.com/yannnz
+[simrandhaliw]: https://github.com/simrandhaliw
+[yih-wang]: https://github.com/yih-wang