From 46c68ab92c8e1c5c0dac13f220ad7dc49787732a Mon Sep 17 00:00:00 2001 From: Sean McGinnis Date: Thu, 18 Apr 2019 11:33:37 -0500 Subject: [PATCH] Fix announce email project names Argument naming ambiguity led to grabbing the wrong value. This updates repo name handling to grab from the right input. Change-Id: I4987fc012a07344cbc4290e33eae5b772270eb07 Signed-off-by: Sean McGinnis --- openstack_releases/cmds/release_notes.py | 4 ++-- openstack_releases/release_notes.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/openstack_releases/cmds/release_notes.py b/openstack_releases/cmds/release_notes.py index 0778c9f1ab..51a46e4680 100644 --- a/openstack_releases/cmds/release_notes.py +++ b/openstack_releases/cmds/release_notes.py @@ -23,7 +23,7 @@ def main(): formatter_class=argparse.RawDescriptionHelpFormatter) parser.add_argument("repo", metavar='path', action="store", help="repository directory, for example" - " 'openstack/cliff'", + " 'openstack/cliff' or './'", ) parser.add_argument('repo_name', action='store', help='The name of the repository being released', @@ -107,7 +107,7 @@ def main(): repo_path = os.path.abspath(args.repo) notes = release_notes.generate_release_notes( - repo=args.repo, + repo=args.repo_name, repo_path=repo_path, start_revision=args.start_revision, end_revision=args.end_revision, diff --git a/openstack_releases/release_notes.py b/openstack_releases/release_notes.py index 25cb3e0ba2..197e8418f7 100644 --- a/openstack_releases/release_notes.py +++ b/openstack_releases/release_notes.py @@ -376,7 +376,7 @@ def generate_release_notes(repo, repo_path, # release-announce@lists.openstack.org ML (except for # release-test and release candidates) email_to = 'release-announce@lists.openstack.org' - if repo == 'openstack-release-test': + if repo_name == 'openstack-release-test': email_to = 'release-job-failures@lists.openstack.org' elif is_release_candidate: email_to = 'openstack-discuss@lists.openstack.org'