From 23d91449dcd4529959cd58359f1fbaea32ecc1c0 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Oct 09 2020 15:23:03 +0000 Subject: When querying PDC catch and log exceptions so we know more what's going on Signed-off-by: Pierre-Yves Chibon --- diff --git a/toddlers/plugins/pdc_import_compose.py b/toddlers/plugins/pdc_import_compose.py index 809237c..3b78805 100644 --- a/toddlers/plugins/pdc_import_compose.py +++ b/toddlers/plugins/pdc_import_compose.py @@ -175,23 +175,37 @@ class PDCImportCompose(ToddlerBase): # https://github.com/product-definition-center/product-definition-center/issues/228 # https://pdc.fedoraproject.org/rest_api/v1/compose-images/ - pdc["compose-images"]._( - { - "release_id": release_id, - "composeinfo": composeinfo, - "image_manifest": images, - } - ) - - # https://pdc.fedoraproject.org/rest_api/v1/compose-rpms/ - if rpms: - pdc["compose-rpms"]._( + try: + pdc["compose-images"]._( { "release_id": release_id, "composeinfo": composeinfo, - "rpm_manifest": rpms, + "image_manifest": images, } ) + except beanbag.bbexcept.BeanBagException as e: + _log.exception("Failed to import compose-images (BeenBagException): %s", url) + _log.error(e.msg) + _log.error(e.response.url) + _log.error(e.response.text) + raise + + # https://pdc.fedoraproject.org/rest_api/v1/compose-rpms/ + if rpms: + try: + pdc["compose-rpms"]._( + { + "release_id": release_id, + "composeinfo": composeinfo, + "rpm_manifest": rpms, + } + ) + except beanbag.bbexcept.BeanBagException as e: + _log.exception("Failed to import compose-images (BeenBagException): %s", url) + _log.error(e.msg) + _log.error(e.response.url) + _log.error(e.response.text) + raise _log.info("Import done.")