From: Abhishek Lekshmanan <abhishek.lekshmanan@ril.com>
Date: Mon, 21 Sep 2015 16:54:27 +0000 (+0530)
Subject: scripts: release_notes can track original issue
X-Git-Tag: v10.0.0~127^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=5e2c66557e4efe1e6ecbb21adbfa4612f2240fd6;p=ceph.git

scripts: release_notes can track original issue

ceph_release_notes helper now has `--original-issue` (default) and
`--no-original-issue` flags added. This is useful
in backport tracker as the original upstream issue generally tends to be
more informative than the backport issue reference number in the release
notes. Since turning on this flag adds the additional network query of
redmine tracker for each issue, it is turned off by default

Signed-off-by: Abhishek Lekshmanan <abhishek.lekshmanan@ril.com>
---

diff --git a/src/script/ceph-release-notes b/src/script/ceph-release-notes
index 007054d52f0..0075c9d4a33 100755
--- a/src/script/ceph-release-notes
+++ b/src/script/ceph-release-notes
@@ -6,7 +6,7 @@
 
   virtualenv v
   source v/bin/activate
-  pip install githubpy GitPython
+  pip install githubpy GitPython requests
 
 Generate a github access token; this is needed as the anonymous access
 to Github's API will easily hit the limit even with a single invocation.
@@ -29,6 +29,7 @@ import github
 import os
 import re
 import sys
+import requests
 
 from git import Repo
 
@@ -37,9 +38,9 @@ merge_re = re.compile("Merge pull request #(\d+).*")
 fixes_re = re.compile(r"Fixes\:? #(\d+)")
 tracker_re = re.compile("http://tracker.ceph.com/issues/(\d+)")
 signed_off_re = re.compile("Signed-off-by: (.+) <")
+tracker_uri = "http://tracker.ceph.com/issues/{0}.json"
 
-
-def make_release_notes(gh, repo, ref, plaintext):
+def make_release_notes(gh, repo, ref, plaintext,original_issue):
 
     issue2prs = {}
 
@@ -62,6 +63,24 @@ def make_release_notes(gh, repo, ref, plaintext):
                 print ("ERROR: http://github.com/ceph/ceph/pull/" + str(number) + " has no associated issue")
                 continue    
 
+            if original_issue:
+                r = requests.get(tracker_uri.format(issue),params={"include":"relations"})
+
+                # Verify that we are indeed looking at a backports issue
+                if r.json()["issue"]["tracker"]["name"] != "Backport":
+                    continue
+
+                # Now we can get an original issue only if the
+                # relations field is populated, otherwise we fallback
+                # to the tracker number
+                try:
+                    relations = r.json()["issue"]["relations"]
+                except KeyError:
+                    continue
+                related_issues = [str(i['issue_id']) for i in relations if i["relation_type"] == "copied_to"]
+                if related_issues:
+                    issue = ','.join(related_issues)
+
             title = pr['title']
 
             title_re = '^(cli|common|mon|osd|fs|librbd|rbd|fs|mds|objecter|rgw|build/ops|tests|tools|doc|crush|librados):'
@@ -96,9 +115,14 @@ if __name__ == "__main__":
                         help="path to ceph git repo")
     parser.add_argument("--token", default=os.getenv("GITHUB_ACCESS_TOKEN"),
                         help="Github Access Token ($GITHUB_ACCESS_TOKEN otherwise)")
+    parser.add_argument("--original_issue", dest='original_issue', action='store_true',
+                        help="When the issue id is a backport, use original upstream issue id instead (default)")
+    parser.add_argument("--no_original_issue", dest='original_issue', action='store_false',
+                        help="When the issue id is a backport, don't use original upstream issue id instead")
+    parser.set_defaults(original_issue=True)
 
     args = parser.parse_args()
     gh = github.GitHub(
         access_token=args.token)
 
-    make_release_notes(gh, Repo(args.repo), args.rev, args.text)
+    make_release_notes(gh, Repo(args.repo), args.rev, args.text, args.original_issue)