diff options
author | Darragh Bailey <dbailey@hpe.com> | 2016-11-14 13:40:03 +0000 |
---|---|---|
committer | Darragh Bailey <dbailey@hpe.com> | 2016-11-14 13:40:03 +0000 |
commit | bfb18520e6388db63470e43dfee4d2c4cbb0f41e (patch) | |
tree | 349ad2a43c2e50439911e810eaca8d9e4caf2d2f /jenkins_jobs/builder.py | |
parent | 949a3a61e1a303c85ce2595764e141fd19ec7729 (diff) | |
download | python-jenkins-job-builder-bfb18520e6388db63470e43dfee4d2c4cbb0f41e.tar.gz python-jenkins-job-builder-bfb18520e6388db63470e43dfee4d2c4cbb0f41e.tar.xz python-jenkins-job-builder-bfb18520e6388db63470e43dfee4d2c4cbb0f41e.zip |
Ensure keep is an iterable
If no value is provided for 'keep' when calling delete_old_managed,
ensure a default empty iterable is used to prevent an exception.
Change-Id: I3110f8e03e287a6ff067b992ed8631e34121a8d2
Diffstat (limited to 'jenkins_jobs/builder.py')
-rw-r--r-- | jenkins_jobs/builder.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/jenkins_jobs/builder.py b/jenkins_jobs/builder.py index 91881e08..62aad784 100644 --- a/jenkins_jobs/builder.py +++ b/jenkins_jobs/builder.py @@ -150,6 +150,8 @@ class JenkinsManager(object): def delete_old_managed(self, keep=None): jobs = self.get_jobs() deleted_jobs = 0 + if keep is None: + keep = [] for job in jobs: if job['name'] not in keep: if self.is_managed(job['name']): |