From a1e511abda610106bee439c2a2ff22b14b1b063b Mon Sep 17 00:00:00 2001 From: Davanum Srinivas Date: Thu, 30 May 2019 08:41:10 -0400 Subject: [PATCH] Bail out on missing pyyaml dependency Change-Id: Ib1e3b92f8e18ae554f77fede34917d5222c677af --- hack/verify-publishing-bot.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/hack/verify-publishing-bot.py b/hack/verify-publishing-bot.py index 75c22e61a9..b2cf19e0ad 100755 --- a/hack/verify-publishing-bot.py +++ b/hack/verify-publishing-bot.py @@ -20,7 +20,6 @@ import fnmatch import os import sys import json -import yaml def get_gomod_dependencies(rootdir, components): @@ -44,6 +43,7 @@ def get_gomod_dependencies(rootdir, components): def get_rules_dependencies(rules_file): + import yaml with open(rules_file) as f: data = yaml.load(f) return data @@ -59,9 +59,15 @@ def main(): components.sort() rules_file = "/staging/publishing/rules.yaml" - gomod_dependencies = get_gomod_dependencies(rootdir + '/staging/src/k8s.io/', components) + try: + import yaml + except ImportError: + print("Please install missing pyyaml module and re-run %s" % sys.argv[0]) + sys.exit(1) rules_dependencies = get_rules_dependencies(rootdir + rules_file) + gomod_dependencies = get_gomod_dependencies(rootdir + '/staging/src/k8s.io/', components) + processed_repos = [] for rule in rules_dependencies["rules"]: branch = rule["branches"][0]