From 86f658cb55d7c336de984e81766843c9c2b2798b Mon Sep 17 00:00:00 2001 From: Simon Quigley Date: Sun, 17 May 2020 02:57:00 -0500 Subject: [PATCH] We can get away with being a little less verbose now. --- ci/jobgenerator.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/ci/jobgenerator.py b/ci/jobgenerator.py index 07851a4..3ae2dbc 100755 --- a/ci/jobgenerator.py +++ b/ci/jobgenerator.py @@ -160,7 +160,6 @@ class Generator: package_config = "" for text in f.readlines(): package_config += text - print(package_config) else: package_config = template.render(PACKAGING_URL=url, PACKAGING_BRANCH_U=u_branch, @@ -205,7 +204,6 @@ class Generator: return jobs - def create_jenkins_jobs(self): """Interface with Jenkins to create the jobs required @@ -237,7 +235,7 @@ class Generator: # TODO: This is duplicate code, and it should be consolidated if job_name in jobs: job = server.get_job(job_name) - print(job.update_config(package_config, full_response=True)) + job.update_config(package_config) else: job = server.create_job(job_name, str(package_config)) if "merger" in server.views: @@ -259,7 +257,7 @@ class Generator: package) if job_name in jobs: job = server.get_job(job_name) - print(job.update_config(str(package_config), full_response=True)) + job.update_config(str(package_config)) else: job = server.create_job(job_name, str(package_config)) @@ -280,7 +278,7 @@ class Generator: job_name = "mgmt_build_" + release + "_" + jobtype if job_name in jobs: job = server.get_job(job_name) - print(job.update_config(package_config, full_response=True)) + job.update_config(package_config) else: job = server.create_job(job_name, str(package_config)) @@ -293,7 +291,7 @@ class Generator: # Generate one last merger management job if "merger" in jobs: job = server.get_job("merger") - print(job.update_config(package_config, full_response=True)) + job.update_config(package_config) else: job = server.create_job("merger", str(package_config))