Merge pull request #20 from Nordix/update_all_owners

update the all-owners.sh and the ALL-OWNERS

Author: metal3-io-bot

.gitignore

@@ -0,0 +1,2 @@
+*.git
+*.csv

maintainers/ALL-OWNERS

@@ -1,20 +1,23 @@
 # All approvers from all top-level OWNERS files, including OWNERS_ALIASES
 # Generated by community/maintainers/all-owners.sh
+# Each line represents an individual maintainer and the line has to
+# contain the name, affiliation, GitHub handle and the e-mail of the
+# maintainer.
 
 approvers:
-- adilGhaffarDev
-- bfournie
-- derekhiggins
-- dtantsur
-- elfosardo
-- honza
-- hroyrh
-- iurygregory
-- kashifest
-- lentzi90
-- mboukhalfa
-- Rozzii
-- smoshiur1237
-- Sunnatillo
-- tuminoid
-- zaneb
+- Muhammad Adil Ghaffar,Ericsson Software Technology,adilGhaffarDev,[email protected]
+- Bob Fournier,Red Hat,bfournie,[email protected]
+- Derek Higgins,Red Hat,derekhiggins,[email protected]
+- Dmitry Tantsur,Red Hat,dtantsur,[email protected]
+- Riccardo Pittau,Red Hat,elfosardo,[email protected]
+- Honza Pokorny,Red Hat,honza,[email protected]
+- Himanshu Roy,Red Hat,hroyrh,[email protected]
+- Iury Gregory Melo Ferreira,Red Hat,iurygregory,[email protected]
+- Kashif Khan,Ericsson Software Technology,kashifest,[email protected]
+- Lennart Jern,Ericsson Software Technology,lentzi90,[email protected]
+- Mohammed Boukhalfa,Ericsson Software Technology,mboukhalfa,[email protected]
+- Adam Rozman,Ericsson Software Technology,Rozzii,[email protected]
+- Moshiur Rahman,Ericsson Software Technology,smoshiur1237,[email protected]
+- Sunnat Samadov,Ericsson Software Technology,Sunnatillo,[email protected]
+- Tuomo Tanskanen,Ericsson Software Technology,tuminoid,[email protected]
+- Zane Bitter,Red Hat,zaneb,[email protected]

maintainers/all-owners.sh

@@ -66,15 +66,54 @@ all_owners_raw()
     done
 }
 
+# Find e-mail address of the committer based on their name and DCO metadata
+find_mail()
+{
+    local mail name
+    name="$1"
+    for repo in "${REPOS[@]}"; do
+        cd "${repo}.git"
+        mail=$(git log --grep="$name" -n 1 | grep "Signed" | sed -e 's/.*<\(.*\)>/\1/g')
+        cd ..
+        if [[ -n "${mail}" ]]; then
+            break
+        fi
+    done
+    echo "${mail}"
+}
+
+# The idea is to match 3 sources of truth against each other, the DCO
+# signatures, the CNCF maintainer spread sheet, the OWNERS and OWNERS_ALIASES
+# files form each repo. The beneficial side effect of such 3 way matching is
+# that anomalies related to DCO, OWNERS files and the spread sheet can be
+# detected and fixed in order to keep them in sync. Manual intervention might
+# be required e.g. in case of folks who have never signed a DCO or represented
+# under different names in different sources.
 echo "# All approvers from all top-level OWNERS files, including OWNERS_ALIASES"
 echo "# Generated by community/maintainers/all-owners.sh"
+echo "# Each line represents an individual maintainer and the line has to"
+echo "# contain the name, affiliation, GitHub handle and the e-mail of the"
+echo "# maintainer."
+# Download the CNCF maintainer doc and the repository metadata (not the full repo)
+curl "${CURL_OPTS[@]}" "https://raw.githubusercontent.com/cncf/foundation/main/project-maintainers.csv" > "project-maintainers.csv"
+for repo in "${REPOS[@]}"; do
+    git clone --quiet --bare "https://github.com/metal3-io/${repo}"
+done
 
 # remove maintainers groups
 for owner in "${OWNER_TYPES[@]}"; do
     echo -e "\n${owner}:"
-    all_owners_raw "${owner}" | \
+    maintainers=$(all_owners_raw "${owner}" | \
         grep -v -- "-maintainers" | \
         grep -v "null" | \
         grep -v "\.\.\." | \
-        sort -uf
+        sort -uf | sed -e 's/- //g')
+    mapfile -t < <(echo "${maintainers[@]}")
+    for maintainer in "${MAPFILE[@]}"; do
+        # match the GitHub handle against the CNCF maintainer file
+        # remove the double ',' and the windows carrige return from the lines
+        individual="$(grep "${maintainer}" "project-maintainers.csv" | sed -e 's/,,//g' | sed -e 's/\r//g')"
+        mail=$(find_mail "${individual%%,*}")
+        echo "- ${individual}${mail}"
+    done
 done