Break out comments with a role restriction

Author: andrewnicols

tracker_automations/continuous_manage_queues/continuous_manage_queues.sh

@@ -105,7 +105,7 @@ fi
 
 # Behaviour A, before the release (normally the 5 weeks between freeze and release).
 echo "Current time period is ${behaviorAB} the release, lastweekdate is ${lastweekdate}"
-echo "Current behaviour is ${behaviourA3}"
+echo "Current behaviour is ${behaviorA3}"
 
 if [ $behaviorAB == "before" ]; then
     # A1, add the "integration_held" + standard comment to any new feature or improvement arriving to candidates.

tracker_automations/continuous_manage_queues/lib.sh

@@ -107,9 +107,13 @@ function run_A2() {
                    --field "${customfield_currentlyInIntegration}"=Yes \
                    --field "${customfield_componentLeadReview}"=No \
                    --field "${customfield_integrator}"= \
-                   --field "${customfield_tester}"= \
+                   --field "${customfield_tester}"=
+
+        ${basereq} --action addComment \
+                   --issue ${issue} \
                    --comment "Continuous queues manage: Moving to current because it's important" \
                    --role "Integrators"
+
         echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} moved to current: important" >> "${logfile}"
     done
 }
@@ -179,7 +183,10 @@ function run_A3a() {
                        --field "${customfield_currentlyInIntegration}"=Yes \
                        --field "${customfield_componentLeadReview}"=No \
                        --field "${customfield_integrator}"= \
-                       --field "${customfield_tester}"= \
+                       --field "${customfield_tester}"=
+
+            ${basereq} --action addComment \
+                       --issue ${issue} \
                        --comment "Continuous queues manage: Moving to current given we are below the threshold ($currentmin)" \
                        --role "Integrators"
             echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} moved to current: threshold (before ${lastweekdate})" >> "${logfile}"
@@ -287,7 +294,10 @@ function run_B1b() {
                        --field "${customfield_currentlyInIntegration}"=Yes \
                        --field "${customfield_componentLeadReview}"=No \
                        --field "${customfield_integrator}"= \
-                       --field "${customfield_tester}"= \
+                       --field "${customfield_tester}"=
+
+            ${basereq} --action addComment \
+                       --issue ${issue} \
                        --comment "Continuous queues manage: Moving to current given we are below the threshold ($currentmin)" \
                        --role "Integrators"
             echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} moved to current on-sync: threshold" >> "${logfile}"
@@ -399,7 +409,10 @@ function run_C() {
                    --transition "CI Global Self-Transition" \
                    --field "${customfield_currentlyInIntegration}"= \
                    --field "${customfield_integrator}"= \
-                   --field "${customfield_tester}"= \
+                   --field "${customfield_tester}"=
+
+        ${basereq} --action addComment \
+                   --issue ${issue} \
                    --comment "Continuous queues manage: Moving out from current because it's held" \
                    --role "Integrators"
         echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} moved out from current: held" >> "${logfile}"

tracker_automations/normal_manage_queues/lib.sh

@@ -69,7 +69,10 @@ function run_A() {
                    --field "${customfield_currentlyInIntegration}"=Yes \
                    --field "${customfield_componentLeadReview}"=No \
                    --field "${customfield_integrator}"= \
-                   --field "${customfield_tester}"= \
+                   --field "${customfield_tester}"=
+
+        ${basereq} --action addComment \
+                   --issue ${issue} \
                    --comment "Normal queues manage: Moving to current because it's important" \
                    --role "Integrators"
         echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} moved to current: important" >> "${logfile}"
@@ -141,7 +144,10 @@ function run_B() {
                     --field "${customfield_currentlyInIntegration}"=Yes \
                     --field "${customfield_componentLeadReview}"=No \
                     --field "${customfield_integrator}"= \
-                    --field "${customfield_tester}"= \
+                    --field "${customfield_tester}"=
+
+            ${basereq} --action addComment \
+                       --issue ${issue} \
                        --comment "Normal queues manage: Moving to current given we are below the threshold ($currentmin)" \
                        --role "Integrators"
             echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} moved to current: threshold" >> "${logfile}"
@@ -179,7 +185,10 @@ function run_C() {
         ${basereq} --action transitionIssue \
                    --issue ${issue} \
                    --transition "CI Global Self-Transition" \
-                   --field "${customfield_integrationPriority}"=1 \
+                   --field "${customfield_integrationPriority}"=1
+
+        ${basereq} --action addComment \
+                   --issue ${issue} \
                    --comment "Normal queues manage: Raising integration priority after ${waitingdays} days awaiting" \
                    --role "Integrators"
         echo "$BUILD_NUMBER $BUILD_TIMESTAMP ${issue} raised integration priority" >> "${logfile}"