From 106177bf531cf96cbfe4c1ba8519b6bc52d7e4ec Mon Sep 17 00:00:00 2001 From: Dan Callaghan Date: May 14 2018 00:49:25 +0000 Subject: fix message consumers to handle missing test_case_name attribute The RemoteOriginalSpecNvrRule rules do not have a test_case_name attribute anymore. --- diff --git a/greenwave/consumers/resultsdb.py b/greenwave/consumers/resultsdb.py index a297ccf..13b4810 100644 --- a/greenwave/consumers/resultsdb.py +++ b/greenwave/consumers/resultsdb.py @@ -129,7 +129,7 @@ class ResultsDBHandler(fedmsg.consumers.FedmsgConsumer): applicable_policies = set() for policy in config['policies']: for rule in policy.rules: - if rule.test_case_name == testcase: + if getattr(rule, 'test_case_name', None) == testcase: applicable_policies.add(policy) log.debug("messaging: found %i applicable policies of %i for testcase %r", len(applicable_policies), len(config['policies']), testcase) diff --git a/greenwave/consumers/waiverdb.py b/greenwave/consumers/waiverdb.py index de1e28a..3c0e6ff 100644 --- a/greenwave/consumers/waiverdb.py +++ b/greenwave/consumers/waiverdb.py @@ -66,7 +66,7 @@ class WaiverDBHandler(fedmsg.consumers.FedmsgConsumer): testcase = msg['testcase'] for policy in config['policies']: for rule in policy.rules: - if rule.test_case_name == testcase: + if getattr(rule, 'test_case_name', None) == testcase: data = { 'decision_context': policy.decision_context, 'product_version': product_version,