yahoo-eng-team team mailing list archive

[Kun Huang <gareth@xxxxxxxxxxxxxxx>]

Public bug reported:

The function exception_to_str() has a history
https://review.openstack.org/#/c/37421/ and it makes:

LOG.warn('The bug is %s: ' % e)

to

LOG.warn(The bug is %s: ' % exception_to_str(e))

It's not so concise, because the log message writer would care about
python encoding issues when they writing something. We could move the
behavior of using this function to GlanceException.__init__, and more
codes could keep better readability. And the best benefit in mind is
that once we change original ideas
https://review.openstack.org/#/c/37421/ someday, we could only fix a
small amount of codes.

** Affects: glance
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Glance.
https://bugs.launchpad.net/bugs/1419588

Title:
  move  exception_to_str behavior to common place

Status in OpenStack Image Registry and Delivery Service (Glance):
  New

Bug description:
  The function exception_to_str() has a history
  https://review.openstack.org/#/c/37421/ and it makes:

  LOG.warn('The bug is %s: ' % e)

  to

  LOG.warn(The bug is %s: ' % exception_to_str(e))

  It's not so concise, because the log message writer would care about
  python encoding issues when they writing something. We could move the
  behavior of using this function to GlanceException.__init__, and more
  codes could keep better readability. And the best benefit in mind is
  that once we change original ideas
  https://review.openstack.org/#/c/37421/ someday, we could only fix a
  small amount of codes.

To manage notifications about this bug go to:
https://bugs.launchpad.net/glance/+bug/1419588/+subscriptions