chore(pdcli): make incident:assign command work

see martindstone/pagerduty-cli#18 for details
georgettica · Oct 21, 2021 · 9e17774 · 9e17774
1 parent 31d2d7b
commit 9e17774
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/utils/bashrc.d/10-pdcli.bashrc b/utils/bashrc.d/10-pdcli.bashrc
@@ -17,7 +17,8 @@ function pdsr() {
 # pdstst will silent-test the incident id provided partially (still need to merge)
 function pdstst() {
   local INCIDENTID=$1
-  pd incident:assign -u $SILENT_TEST_USER_ID -i $INCIDENTID
+  # TODO: once https://github.com/martindstone/pagerduty-cli/issues/18 is solved, revert to using `-u`?
+  pd incident:assign --assign_to_user_ids $SILENT_TEST_USER_ID -i $INCIDENTID
   echo "need functionality to merge the incident to the $LONG_RUNNING_INCIDENT_ID, so far just opening it manually and merging via UI"
   pd incident:open -i $LONG_RUNNING_INCIDENT_ID # long term incident to merge into
 }

diff --git a/utils/bin/chgm b/utils/bin/chgm
@@ -118,5 +118,6 @@ fi
 pd incident:ack -i ${PD_ALERT}
 osdctl servicelog post -t ${SL_URL} -p CLUSTER_UUID=${CLUSTER_UUID}
 pd incident:notes -i ${PD_ALERT} -n "$PD_CHGM_NOTE"
-pd incident:assign -i ${PD_ALERT} -u $PD_SILENT_TEST_USER
+# TODO: once https://github.com/martindstone/pagerduty-cli/issues/18 is solved, revert to using `-u`?
+pd incident:assign -i ${PD_ALERT} --assign_to_user_ids=$PD_SILENT_TEST_USER
 pd incident:merge -i ${PD_ALERT} -I ${CHGM_PARENT_INCIDENT}