Update sync_org.sh to provide commands to dump the org. (kubeflow#79)

* Add greynutw, yuzisun, and joeliedtke. THey were already added to the org but not the YAML file.
arrikto · Apr 8, 2019 · d5455c2 · d5455c2
1 parent fda5eaf
commit d5455c2
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 10 deletions.
diff --git a/github-orgs/kubeflow/org.yaml b/github-orgs/kubeflow/org.yaml
@@ -78,6 +78,7 @@ orgs:
         - gaocegege
         - gaoning777
         - garganubhav
+        - greynutw
         - gsunner
         - gyliu513
         - hamelsmu
@@ -93,6 +94,7 @@ orgs:
         - Jimexist
         - jinchihe
         - jinxingwang
+        - joeliedtke
         - johnugeorge
         - jose5918
         - jzp1025
@@ -170,6 +172,7 @@ orgs:
         - yph152
         - YujiOshima
         - yupbank
+        - yuzisun
         - zhenghuiwang
         - ziamsyed
         - zjj2wry

diff --git a/github-orgs/sync_org.sh b/github-orgs/sync_org.sh
@@ -28,10 +28,13 @@ parseArgs() {
   done
 }
 
+
+action=$1
+shift
 parseArgs $*
 
 usage() {
-	echo "sync_org.sh --test_infra_dir=<kubernetes-test-infra-dir> --token_file=<path-to-github-token> [--confirm] [--admins]"
+	echo "sync_org.sh <command> --test_infra_dir=<kubernetes-test-infra-dir> --token_file=<path-to-github-token> [--confirm] [--admins]"
 }
 
 if [ -z ${test_infra_dir} ]; then
@@ -55,12 +58,21 @@ if ${admins}; then
 	FIX_ADMINS=--fix-org-members
 fi
 
-bazel run //prow/cmd/peribolos -- --fix-org-members ${FIX_ADMINS} --config-path ${DIR}/kubeflow/org.yaml \
-	--github-token-path ${token_file} \
-	--required-admins=jlewi \
-	--required-admins=abhi-g \
-	--required-admins=google-admin \
-	--required-admins=googlebot \
-	--required-admins=richardsliu \
-	--required-admins=vicaire \
-	--confirm=${confirm}
+if [ "${action}" == "dump" ]; then
+	#bazel run //prow/cmd/peribolos -- --help
+	bazel run //prow/cmd/peribolos -- --dump=kubeflow \
+		--github-token-path ${token_file} > --config-path ${DIR}/kubeflow/org.yaml
+elif [ "${action}" == "sync" ]; then
+	bazel run //prow/cmd/peribolos -- --fix-org-members ${FIX_ADMINS} --config-path ${DIR}/kubeflow/org.yaml \
+		--github-token-path ${token_file} \
+		--required-admins=jlewi \
+		--required-admins=abhi-g \
+		--required-admins=google-admin \
+		--required-admins=googlebot \
+		--required-admins=richardsliu \
+		--required-admins=vicaire \
+		--confirm=${confirm}
+else 
+  echo "command=${action} is not a valid command; valid commands are dump and sync"
+  exit 1
+fi