Open Stack

Mike,

Thanks for pointing this out.  I thought I had gotten all of these and
this supports the need for the hacking check I will work on.

We also hit errors like this on our internal CI due to a bad merge.

Anyway, I am on top of this.  Will fix asap.

Jay

On Sun, 2014-07-20 at 15:49 -0700, Mike Perez wrote:
> Recent change to how we use gettextutils [1] seem to not import _
> anymore. According to the commit, enable_lazy() is suppose to
> *replace* gettextutils.install(), but I can't see where in the code
> that happens or see it actually working.
> 
> I have reported a bug [2] that cinder-manage and cinder-rtstool [3] no
> longer work as a result. Using gettextutils.install() seems to resolve
> the issue or explicitly import _. Can someone who is familiar with
> this module please enlighten me how this is suppose to work?
> 
> [1] - https://review.openstack.org/#/c/105561/4
> [2] - https://bugs.launchpad.net/cinder/+bug/1345789
> [3] - https://bugs.launchpad.net/cinder/+bug/1345789/comments/4
> 
> --
> Mike Perez