diff options
| -rwxr-xr-x | kolla/image/build.py | 4 | ||||
| -rw-r--r-- | kolla/tests/test_build.py | 8 |
2 files changed, 6 insertions, 6 deletions
diff --git a/kolla/image/build.py b/kolla/image/build.py index 84fc32b01..3cfbbc202 100755 --- a/kolla/image/build.py +++ b/kolla/image/build.py @@ -1168,10 +1168,10 @@ class KollaWorker(object): LOG.info("Images that failed to build") LOG.info("===========================") for name, status in sorted(self.image_statuses_bad.items()): - LOG.error('%s Failed with status: %s', name, status) + LOG.error('%s Failed with status: %s', name, status.value) results['failed'].append({ 'name': name, - 'status': status, + 'status': status.value, }) if self.conf.logs_dir and status == Status.ERROR: linkname = os.path.join(self.conf.logs_dir, diff --git a/kolla/tests/test_build.py b/kolla/tests/test_build.py index 2812e115e..00bdd6d32 100644 --- a/kolla/tests/test_build.py +++ b/kolla/tests/test_build.py @@ -646,11 +646,11 @@ class KollaWorkerTest(base.TestCase): def test_summary(self): kolla = build.KollaWorker(self.conf) kolla.images = self.images - kolla.image_statuses_good['good'] = None - kolla.image_statuses_bad['bad'] = None - kolla.image_statuses_unmatched['unmatched'] = None + kolla.image_statuses_good['good'] = build.Status.BUILT + kolla.image_statuses_bad['bad'] = build.Status.ERROR + kolla.image_statuses_unmatched['unmatched'] = build.Status.UNMATCHED results = kolla.summary() - self.assertIsNone(results['failed'][0]['status']) + self.assertEqual('error', results['failed'][0]['status']) @mock.patch('shutil.copytree') def test_work_dir(self, copytree_mock): |