I hope you're aware that the jiraIssue.startsWith("M") check isn't good
enough...
https://cwiki.apache.org/confluence/display/MAVEN/Maven+JIRA+issues+overview
https://issues.apache.org/jira/secure/BrowseProjects.jspa?selectedCategory=10510
thanks,
Robert
On Mon, 08 Jan 2018 11:03:30 +0100, <[email protected]> wrote:
This is an automated email from the ASF dual-hosted git repository.
stephenc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-jenkins-env.git
The following commit(s) were added to refs/heads/master by this push:
new 88aff47 Stop commenting on INFRA tickets
88aff47 is described below
commit 88aff478d3b9c821077ebc4338f5ee90f9c56970
Author: Stephen Connolly <[email protected]>
AuthorDate: Mon Jan 8 10:03:24 2018 +0000
Stop commenting on INFRA tickets
---
vars/jenkinsNotify.groovy | 20 +++++++++++++-------
1 file changed, 13 insertions(+), 7 deletions(-)
diff --git a/vars/jenkinsNotify.groovy b/vars/jenkinsNotify.groovy
index 99e3ce4..24c34a3 100644
--- a/vars/jenkinsNotify.groovy
+++ b/vars/jenkinsNotify.groovy
@@ -64,21 +64,27 @@ def call(Map params = [:]) {
// comment on any jira tickets
def jiraIssues = null
+ def jiraMavens = []
try {
jiraIssues = jiraIssueSelector(issueSelector: [$class:
'DefaultIssueSelector'])
for (def jiraIssue in jiraIssues) {
- try {
- jiraComment body:
"${messageSubject}\n\n${messageBody}", issueKey: jiraIssue
- } catch (e) {
- echo "WARNING: Could not update ${jiraIssue}:
${e.message}"
- }
+ // only comment on maven's issues, all our trackers start
with M
+ // may end up commenting on other TLPs in some cases but
should be very rare
+ if (jiraIssue.startsWith("M")) {
+ try {
+ jiraComment body:
"${messageSubject}\n\n${messageBody}", issueKey: jiraIssue
+ } catch (e) {
+ echo "WARNING: Could not update ${jiraIssue}:
${e.message}"
+ }
+ jiraMavens += jiraIssue
+ }
}
} catch (e) {
echo "WARNING: Could not determine JIRA issues: ${e.message}"
}
// set the build description to the jira ticket id's
- if (jiraIssues != null && !jiraIssues.empty) {
- currentBuild.description = "${jiraIssues.join(', ')}"
+ if (!jiraMavens.empty) {
+ currentBuild.description = "${jiraMavens.join(', ')}"
}
// add the changes to the email
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
