Change perf test trigger phases
authorMichelle McDaniel <adiaaida@gmail.com>
Wed, 8 Mar 2017 18:54:29 +0000 (10:54 -0800)
committerMichelle McDaniel <adiaaida@gmail.com>
Wed, 8 Mar 2017 18:54:29 +0000 (10:54 -0800)
Currently the trigger phases for perf testing use an underscore between OS
and architecture. This is inconsistent with our other trigger phrases.
This change changes the underscore to whitespace.

Commit migrated from https://github.com/dotnet/coreclr/commit/a80e0eed648eb1f4d82d2bdceb9a2a9b03ce41ca

src/coreclr/perf.groovy

index 3bd9dcc..1513217 100644 (file)
@@ -99,7 +99,7 @@ def static getOSGroup(def os) {
                                TriggerBuilder builder = TriggerBuilder.triggerOnPullRequest()
                                builder.setGithubContext("${os} ${arch} CoreCLR Perf Tests")
                                builder.triggerOnlyOnComment()
-                               builder.setCustomTriggerPhrase("(?i).*test\\W+${os}_${arch}\\W+perf.*")
+                               builder.setCustomTriggerPhrase("(?i).*test\\W+${os}\\W+${arch}\\W+perf.*")
                                builder.triggerForBranch(branch)
                                builder.emitTrigger(newJob)
                        }
@@ -179,7 +179,7 @@ def static getOSGroup(def os) {
                                TriggerBuilder builder = TriggerBuilder.triggerOnPullRequest()
                                builder.setGithubContext("${os} ${arch} CoreCLR Throughput Perf Tests")
                                builder.triggerOnlyOnComment()
-                               builder.setCustomTriggerPhrase("(?i).*test\\W+${os}_${arch}\\W+throughput.*")
+                               builder.setCustomTriggerPhrase("(?i).*test\\W+${os}\\W+${arch}\\W+throughput.*")
                                builder.triggerForBranch(branch)
                                builder.emitTrigger(newJob)
                        }