<div dir="ltr">This change was recently merged:<div><br></div><div><a href="https://review.openstack.org/#/c/69133/">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><div><div><br></div>-- <br><div dir="ltr"><div>IRC: radix</div>Christopher Armstrong</div>
</div></div>