From 9e3efc3fb47612c0904ba126d0fc9fa03c947808 Mon Sep 17 00:00:00 2001 From: Ralph Bean Date: Fri, 15 Jul 2016 13:06:26 -0400 Subject: [PATCH] Remove a bunch of unnecessary stuff. This is because get_module() is already called *inside* the call to get_module_tag() and get_module_dependencies(). --- rida/pdc.py | 1 - rida/scheduler/handlers/modules.py | 8 ++------ 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/rida/pdc.py b/rida/pdc.py index 6d85e0e8..4df62101 100644 --- a/rida/pdc.py +++ b/rida/pdc.py @@ -96,7 +96,6 @@ def get_variant_dict(data): return result - def variant_dict_from_str(module_str): """ :param module_str: a string to match in PDC diff --git a/rida/scheduler/handlers/modules.py b/rida/scheduler/handlers/modules.py index de25b72f..d383c991 100644 --- a/rida/scheduler/handlers/modules.py +++ b/rida/scheduler/handlers/modules.py @@ -42,14 +42,10 @@ def init(config, session, msg): build = rida.database.ModuleBuild.from_fedmsg(session, msg) pdc_session = rida.pdc.get_pdc_client_session(config) - build_data = build.json() - log.debug("Getting module from pdc with input_data=%s" % build_data) - module_info = rida.pdc.get_module(pdc_session, build_data) - + module_info = build.json() log.debug("Received module_info=%s from pdc" % module_info) - tag = rida.pdc.get_module_tag(pdc_session, module_info) - log.info("Found tag=%s for module %s-%s-%s" % (tag, build['name'], build['version'], build['release'])) + log.info("Found tag=%s for module %r" % (tag, build)) dependencies = rida.pdc.get_module_dependencies(pdc_session, module_info) builder = rida.builder.KojiModuleBuilder(build.name, config)