Merge remote-tracking branch 'origin/candidate-9.4.x'

Signed-off-by: Jake Smith <[email protected]> # Conflicts: # commons-hpcc/pom.xml # dfsclient/pom.xml # pom.xml # wsclient/pom.xml
hpcc-systems · Feb 22, 2024 · 765748c · 765748c
2 parents 23bf3dd + cd7229b
commit 765748c
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 27 deletions.
diff --git a/.github/workflows/Jirabot.yml b/.github/workflows/Jirabot.yml
@@ -37,13 +37,15 @@ jobs:
           PULL_URL: ${{ github.event.pull_request.html_url }}
           COMMENTS_URL: ${{ github.event.pull_request.comments_url }}
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
+          GHUB_JIRA_USER_MAP: ${{ vars.GHUB_JIRA_USER_MAP }}
+          JIRA_ISSUE_PROPERTY_MAP: ${{ vars.JIRA_ISSUE_PROPERTY_MAP }}
         run: |
             import os
             import re
+            import json
             from jira.client import JIRA
 
-            def updateIssue(jira, issue, user: str, pull_url: str) -> str:
+            def updateIssue(jira, issue, prAuthor: str, propertyMap: dict, pull_url: str) -> str:
                 result = ''
 
                 statusName = str(issue.fields.status)
@@ -66,18 +68,26 @@ jobs:
                         transitions = jira.transitions(issue)
                         result += 'Error: Transition: "' + transition + '" failed with: "' + str(error) + '" Valid transitions=' + str(transitions) + '\n'
 
-                if issue.fields.customfield_10010 is None:
-                    issue.update(fields={'customfield_10010': pull_url})
+                prFieldName = propertyMap.get('pullRequestFieldName', 'customfield_10010')
+                try:
+                    currentPR = getattr(issue.fields, prFieldName)
+                except:
+                    currentPR = None
+                    print('Error: Unable to get current pull request with field name: ' + prFieldName)
+
+                if currentPR is None:
+                    issue.update(fields={prFieldName: pull_url})
                     result += 'Updated PR\n'
-                elif issue.fields.customfield_10010 is not None and issue.fields.customfield_10010 != pull_url:
+                elif currentPR is not None and currentPR != pull_url:
                     result += 'Additional PR: ' + pull_url + '\n'
 
-                if issue.fields.assignee is None:
-                    jira.assign_issue(issue, user)
-                    result += 'Assigning user: ' + user + '\n'
-                elif issue.fields.assignee is not None and issue.fields.assignee.name.lower() != user.lower():
-                    result += 'Changing assignee from: ' + issue.fields.assignee.name + ' to: ' + user + '\n'
-                    jira.assign_issue(issue, user)
+                if prAuthor:
+                    if issue.fields.assignee is None:
+                        jira.assign_issue(issue, prAuthor)
+                        result += 'Assigning user: ' + prAuthor + '\n'
+                    elif issue.fields.assignee is not None and issue.fields.assignee.name.lower() != prAuthor.lower():
+                        result += 'Changing assignee from: ' + issue.fields.assignee.name + ' to: ' + prAuthor + '\n'
+                        jira.assign_issue(issue, prAuthor)
 
                 return result
 
@@ -86,37 +96,50 @@ jobs:
             jira_url = os.environ['JIRA_URL']
             pr = os.environ['PULL_REQUEST_NUMBER']
             title = os.environ['PULL_REQUEST_TITLE']
-            user = os.environ['PULL_REQUEST_AUTHOR_NAME']
+            prAuthor = os.environ['PULL_REQUEST_AUTHOR_NAME']
             pull_url = os.environ['PULL_URL']
             github_token = os.environ['GITHUB_TOKEN']
             comments_url = os.environ['COMMENTS_URL']
 
-            print("%s %s %s" % (title, user, comments_url))
+            print("%s %s %s" % (title, prAuthor, comments_url))
             result = ''
             issuem = re.search("(HPCC4J|JAPI)-[0-9]+", title)
             if issuem:
                 nameCorrectionPattern = re.compile("hpcc4j", re.IGNORECASE)
                 issue_name = nameCorrectionPattern.sub("JAPI",issuem.group())
 
-                userDict = {
-                    'kunalaswani': 'kunal.aswani',
-                    'timothyklemm': 'klemti01',
-                    'jpmcmu': 'mcmuja01',
-                    'asselitx': 'terrenceasselin',
-                    'jeclrsg': 'clemje01',
-                    'jackdelv': 'delvecja',
-                }
-                user = userDict.get(user, user)
+                userDict = json.loads(os.environ['GHUB_JIRA_USER_MAP'])
+                if not isinstance(userDict, dict):
+                    userDict = {}
+
+                if prAuthor in userDict:
+                    prAuthor = userDict.get(prAuthor)
+                    print('Mapped Github user to Jira user: ' + prAuthor)
 
                 options = {
                     'server': jira_url
                 }
 
                 jira = JIRA(options=options, basic_auth=(jirabot_user, jirabot_pass))
+
+                # Check if prAuthor exists in Jira
+                try:
+                    jiraUser = jira.user(prAuthor)
+                    if jiraUser is None:
+                        prAuthor = None
+                        print('Error: Unable to find Jira user: ' + prAuthor + ' continuing without assigning')
+                except Exception as error:
+                    prAuthor = None
+                    print('Error: Unable to find Jira user: ' + prAuthor + ' with error: ' + str(error) + ' continuing without assigning')
+
                 issue = jira.issue(issue_name)
                 result = 'Jirabot Action Result:\n'
 
-                result += updateIssue(jira, issue, user, pull_url)
+                jiraIssuePropertyMap = json.loads(os.environ['JIRA_ISSUE_PROPERTY_MAP'])
+                if not isinstance(jiraIssuePropertyMap, dict):
+                    jiraIssuePropertyMap = {}
+
+                result += updateIssue(jira, issue, prAuthor, jiraIssuePropertyMap, pull_url)
                 jira.add_comment(issue, result)
             else:
                 print('Unable to find Jira issue name in title')

diff --git a/.github/workflows/JirabotMerge.yml b/.github/workflows/JirabotMerge.yml
@@ -190,10 +190,6 @@ jobs:
             jirabot_pass = os.environ['JIRABOT_PASSWORD']
             jira_url = os.environ['JIRA_URL']
 
-            if not jira_url:
-                jira_url = 'https://track.hpccsystems.com'
-                print('Jira URL us empty defaulting to: ' + jira_url)
-
             pr = os.environ['PULL_REQUEST_NUMBER']
             title = os.environ['PULL_REQUEST_TITLE']
             user = os.environ['PULL_REQUEST_AUTHOR_NAME']