From 2f0ff3e476c59001b3c5fd172a6fa00056b680b8 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Jan 27 2018 09:38:21 +0000
Subject: Add logging on pagure-ci and catch exceptions


When we fail to trigger a build on jenkins, it raises an exception and
catching it makes it easier to debug and know what's going on.

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure-ci/pagure_ci_server.py b/pagure-ci/pagure_ci_server.py
index 7c9011d..2a55d30 100644
--- a/pagure-ci/pagure_ci_server.py
+++ b/pagure-ci/pagure_ci_server.py
@@ -89,6 +89,7 @@ def handle_messages():
         url = request.project.ci_hook.ci_url.rstrip('/')
 
         if data['ci_type'] == 'jenkins':
+            _log.info('Jenkins CI')
             repo = '%s/%s' % (
                 _config['GIT_URL_GIT'].rstrip('/'),
                 request.project_from.path)
@@ -106,12 +107,16 @@ def handle_messages():
             server = jenkins.Jenkins(base_url)
             _log.info('Triggering at: %s for: %s - data: %s' % (
                 base_url, jenkins_name, data))
-            server.build_job(
-                name=jenkins_name,
-                parameters=data,
-                token=request.project.ci_hook.pagure_ci_token
-            )
-            _log.info('Build triggered')
+            try:
+                server.build_job(
+                    name=jenkins_name,
+                    parameters=data,
+                    token=request.project.ci_hook.pagure_ci_token
+                )
+                _log.info('Build triggered')
+            except Exception as err:
+                _log.info('An error occured: %s', err)
+
         else:
             _log.warning('Un-supported CI type')