Fix triggering of perf correctness runs
authorDrew Scoggins <anscoggi@microsoft.com>
Wed, 3 May 2017 06:55:33 +0000 (23:55 -0700)
committerDrew Scoggins <anscoggi@microsoft.com>
Wed, 3 May 2017 06:55:33 +0000 (23:55 -0700)
When I made the change to have the correctness runs fire in a PR without
the trigger phrase I accidentally did it for all PR's not just for the
smoketest runs.  This change fixes that.

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

src/coreclr/perf.groovy

index b490219..2d09c31 100644 (file)
@@ -123,7 +123,16 @@ def static getOSGroup(def os) {
                 
                 if (isPR) {
                     TriggerBuilder builder = TriggerBuilder.triggerOnPullRequest()
-                    builder.setGithubContext("${os} ${arch} CoreCLR Perf Tests")
+                    if (isSmoketest)
+                    {
+                        builder.setGithubContext("${os} ${arch} CoreCLR Perf Tests Correctness")
+                    }
+                    else
+                    {
+                        builder.setGithubContext("${os} ${arch} CoreCLR Perf Tests")
+                        builder.triggerOnlyOnComment()
+                        builder.setCustomTriggerPhrase("(?i).*test\\W+${os}\\W+${arch}\\W+perf.*")
+                    }
                     builder.triggerForBranch(branch)
                     builder.emitTrigger(newJob)
                 }