diff --git a/advisors/review_tool.py b/advisors/review_tool.py index 4d36c25957c48126cc9cd759e3a93426dd5281d4..dccb2703bf3ceba8ad48f62930ea62925ddab5a7 100755 --- a/advisors/review_tool.py +++ b/advisors/review_tool.py @@ -361,7 +361,7 @@ def check_maintainer_changes(): sigs = {} lst_files = subprocess.getoutput("git diff --name-status remotes/origin/master..") for line in lst_files.splitlines(): - status, item = line.split() + status, item = line.split(maxsplit=1) if status != "M": continue if item.startswith("sig/") and item.endswith("/OWNERS"): @@ -383,7 +383,7 @@ def check_sig_information_changes(): sigs = {} lst_files = subprocess.getoutput("git diff --name-status remotes/origin/master..") for line in lst_files.splitlines(): - status, item = line.split() + status, item = line.split(maxsplit=1) if status != "M": continue if item == "sig/sigs.yaml": diff --git a/prow/prow_review_tool b/prow/prow_review_tool index e2a25d0d85f8f7d0016437f86e02dffe7e80d87c..5334408774cecf50cb4bfcec7e4d3ce0c096bf76 100755 --- a/prow/prow_review_tool +++ b/prow/prow_review_tool @@ -52,8 +52,8 @@ if __name__ == '__main__': return_code = 0 if args.payload: data = json.loads(args.payload) - if args.event == 'Merge Request Hook' and data['action'] == 'open' \ - or data['action'] == 'update': + if args.event == 'Merge Request Hook' and data['action'] == 'open'\ + or (data['action'] == 'update' and data['action_desc'] == "source_branch_changed"): subp = subprocess.run(["python3", get_cmd(), "-u", data['pull_request']['html_url'], "-w", "/tmp/review_dir", "-c"], stdout=subprocess.PIPE, @@ -77,7 +77,7 @@ if __name__ == '__main__': check=False) return_code = subp.returncode else: - print("Event type: %s not support." % args.event) + print("prow_review_tool: this action type not need to process.") if return_code != 0: print(subp.stdout) sys.exit(return_code)