Repair comment regexes on release templates 20/62120/2
authorLott, Christopher (cl778h) <cl778h@att.com>
Thu, 31 Oct 2019 00:27:13 +0000 (20:27 -0400)
committerLott, Christopher (cl778h) <cl778h@att.com>
Thu, 31 Oct 2019 00:31:26 +0000 (20:31 -0400)
Change the comment-added-contains-event string for the release-verify
and release-merge templates to the same regexes used in all other verify
and merge templates, instead of custom versions that didn't work;
verify now uses '^Patch Set\s+\d+:\s+(recheck|reverify)\s*$'
merge now uses  '^Patch Set\s+\d+:\s+remerge\s*$'

Change-Id: I4436568046ffbc15225115726e2664824bd6bea2
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>
jjb/lf-release-jobs.yaml
releasenotes/notes/fix-release-job-comment-triggers-692baf00dabe99e8.yaml [new file with mode: 0644]

index c2bb749..c838482 100644 (file)
@@ -53,8 +53,8 @@
           exclude-no-code-change: false
       - draft-published-event
       - comment-added-contains-event:
-          comment-contains-value: |
-            ^Patch Set[ ]+[0-9]+:([ ]+|[\n]+)(recheck|reverify)$
+          # yamllint disable-line rule:line-length
+          comment-contains-value: '^Patch Set\s+\d+:\s+(recheck|reverify)\s*$'
 
     #####################
     # Job Configuration #
     gerrit_merge_triggers:
       - change-merged-event
       - comment-added-contains-event:
-          comment-contains-value: remerge$
+          comment-contains-value: '^Patch Set\s+\d+:\s+remerge\s*$'
 
     #####################
     # Job Configuration #
diff --git a/releasenotes/notes/fix-release-job-comment-triggers-692baf00dabe99e8.yaml b/releasenotes/notes/fix-release-job-comment-triggers-692baf00dabe99e8.yaml
new file mode 100644 (file)
index 0000000..1687fa6
--- /dev/null
@@ -0,0 +1,8 @@
+---
+fixes:
+  - |
+    Change the comment-added-contains-event strings for the release-verify and
+    release-merge templates to the same regexes used in other verify and merge
+    templates, instead of custom versions that didn't seem to work;
+    verify now uses '^Patch Set\s+\d+:\s+(recheck|reverify)\s*$'
+    merge now uses  '^Patch Set\s+\d+:\s+remerge\s*$'