<div dir="ltr">I've filed a bug about this, <a href="https://bugs.launchpad.net/heat/+bug/1281644">https://bugs.launchpad.net/heat/+bug/1281644</a><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 18, 2014 at 9:15 AM, Christopher Armstrong <span dir="ltr"><<a href="mailto:chris.armstrong@rackspace.com" target="_blank">chris.armstrong@rackspace.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This change was recently merged:<div><br></div><div><a href="https://review.openstack.org/#/c/69133/" target="_blank">https://review.openstack.org/#/c/69133/</a></div>
<div><br></div><div>Unfortunately it didn't enable lazy translations for the unit tests, so it didn't catch the many places in Heat that won't work when lazy translations are enabled. Notably there are a lot of cases where the code adds the result of a call to _() with another string, and Message objects (which are returned from _ when lazy translations are enabled) can't be added, resulting in an exception being raised.</div>

<div><br></div><div>I think the best course of action would be to revert this change, and then reintroduce it along with patches to fix all the problems, while enabling it for the unit tests so bugs won't be reintroduced in the future.</div>

<div><br></div><div>Interestingly it also didn't fail any of the tempest tests, I'm not sure why.</div><span class="HOEnZb"><font color="#888888"><div><div><br></div>-- <br><div dir="ltr"><div>IRC: radix</div>Christopher Armstrong</div>

</div></font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div>IRC: radix</div>Christopher Armstrong<div><br></div></div>
</div></div>