openstack/kolla-ansible
Code Issues Proposed changes

Merge "Log information about successful push only if it really is"

Browse Source
This commit is contained in:
Jenkins 2016-01-07 08:41:39 +00:00 committed by Gerrit Code Review
commit 90e178b57e
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
kolla/cmd/build.py
View File

@ -89,13 +89,14 @@ class PushThread(Thread):
image = self.queue.get() image = self.queue.get()
LOG.debug('%s:Try to push the image', image['name']) LOG.debug('%s:Try to push the image', image['name'])
self.push_image(image) self.push_image(image)
LOG.info('%s:Pushed successfully', image['name'])
except ConnectionError: except ConnectionError:
LOG.exception('%s:Make sure Docker is running and that you' LOG.exception('%s:Make sure Docker is running and that you'
' have the correct privileges to run Docker' ' have the correct privileges to run Docker'
' (root)', image['name']) ' (root)', image['name'])
image['status'] = "connection_error" image['status'] = "connection_error"
finally: finally:
if "error" not in image['status']:
LOG.info('%s:Pushed successfully', image['name'])
self.queue.task_done() self.queue.task_done()
def push_image(self, image): def push_image(self, image):