From fa09dcc8289455aa53fff3d853e906804398f605 Mon Sep 17 00:00:00 2001 From: Drew Scoggins Date: Tue, 2 May 2017 23:55:33 -0700 Subject: [PATCH] Fix triggering of perf correctness runs 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 | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/coreclr/perf.groovy b/src/coreclr/perf.groovy index b490219..2d09c31 100644 --- a/src/coreclr/perf.groovy +++ b/src/coreclr/perf.groovy @@ -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) } -- 2.7.4