From 3375d70b1fc74cca955d4da41e7cc08a9e7e1276 Mon Sep 17 00:00:00 2001 From: Doug Hellmann Date: Tue, 10 Apr 2018 11:17:44 -0400 Subject: [PATCH] be more explicit about the configuration being tested Show the directories being read and the SHA of the patch being tested. Change-Id: Ifa73c15a4cb7d965a397753463fdd2df68ae4acf Signed-off-by: Doug Hellmann --- playbooks/files/project-requirements-change.py | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/playbooks/files/project-requirements-change.py b/playbooks/files/project-requirements-change.py index 802449830f..d82ca15de5 100755 --- a/playbooks/files/project-requirements-change.py +++ b/playbooks/files/project-requirements-change.py @@ -96,12 +96,21 @@ def main(): if reqdir is None: if args.local: + print('selecting default requirements directory for local mode') reqdir = os.path.dirname( os.path.dirname( os.path.dirname(sys.argv[0]))) else: + print('selecting default requirements directory for normal mode') reqdir = _DEFAULT_REQS_DIR + print('Branch: {}'.format(branch)) + print('Source: {}'.format(args.src_dir)) + print('Requirements: {}'.format(reqdir)) + + sha, _ = run_command('git log -n 1 --format=%H') + print('Patch under test: {}'.format(sha)) + # build a list of requirements from the global list in the # openstack/requirements project so we can match them to the changes with tempdir() as reqroot: @@ -115,7 +124,7 @@ def main(): # build a list of requirements in the proposed change, # and check them for style violations while doing so head_proj = project.read(cwd) - head_reqs = check.RequirementsList('HEAD', head_proj) + head_reqs = check.RequirementsList(sha, head_proj) # Don't apply strict parsing rules to stable branches. # Reasoning is: # - devstack etc protect us from functional issues