From f19aafe17b751fcb9ae39d02ff022e22dda8b0af Mon Sep 17 00:00:00 2001 From: Nathan Cutler Date: Fri, 27 Jan 2017 14:01:35 +0100 Subject: [PATCH] tools: ceph-release-notes: ignore low-numbered PRs Fixes: http://tracker.ceph.com/issues/18695 Signed-off-by: Nathan Cutler --- src/script/ceph-release-notes | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/script/ceph-release-notes b/src/script/ceph-release-notes index 77bb9dfbe36..7937e049e0f 100755 --- a/src/script/ceph-release-notes +++ b/src/script/ceph-release-notes @@ -97,6 +97,11 @@ def make_release_notes(gh, repo, ref, plaintext, verbose, strict, use_tags): merge = merge_re.match(commit.summary) if merge: number = merge.group(1) + print ("Considering PR#" + number) + # do not pick up ceph/ceph-qa-suite.git PRs + if int(number) < 1311: + print ("Ignoring low-numbered PR, probably picked up from ceph/ceph-qa-suite.git") + continue pr = gh.repos("ceph")("ceph").pulls(number).get() title = pr['title'] message_lines = commit.message.split('\n') @@ -142,7 +147,7 @@ def make_release_notes(gh, repo, ref, plaintext, verbose, strict, use_tags): continue if strict: - title_re = '^(?:hammer|infernalis|jewel|kraken):\s+(cli|common|mon|msgr|osd|log|librbd|rbd|fs|mds|objecter|rgw|build/ops|tests|tools|cmake|doc|crush|librados)(:.*)' + title_re = '^(?:hammer|infernalis|jewel|kraken):\s+(cephx|cli|common|mon|msgr|osd|log|librbd|rbd|fs|mds|objecter|rgw|build/ops|tests|tools|cmake|doc|crush|librados)(:.*)' match = re.match(title_re, title) if not match: print ("ERROR: http://github.com/ceph/ceph/pull/" + str(number) + " title " + title.encode("utf-8") + " does not match " + title_re) -- 2.47.3