<div dir="ltr">Thank you for the patches. I merged them, released 1.1.0 and proposed [1]<div><br></div><div>Cheers!,</div><div><br></div><div>[1] //<a href="http://review.openstack.org/445884">review.openstack.org/445884</a><br><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 15, 2017 at 10:14 AM, Gorka Eguileor <span dir="ltr"><<a href="mailto:geguileo@redhat.com" target="_blank">geguileo@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 14/03, Ihar Hrachyshka wrote:<br>
> Hi all,<br>
><br>
> the patch that started to produce log index file for logstash [1] and<br>
> the patch that switched metadata proxy to haproxy [2] landed and<br>
> together busted the functional job because the latter produces log<br>
> messages with null-bytes inside, while os-log-merger is not resilient<br>
> against it.<br>
><br>
> If functional job would be in gate and not just in check queue, that<br>
> would not happen.<br>
><br>
> Attempt to fix the situation in multiple ways at [3]. (For<br>
> os-log-merger patches, we will need new release and then bump the<br>
> version used in gate, so short term neutron patches seem more viable.)<br>
><br>
> I will need support from both authors of os-log-merger as well as<br>
> other neutron members to unravel that. I am going offline in a moment,<br>
> and hope someone will take care of patches up for review, and land<br>
> what's due.<br>
><br>
> [1] <a href="https://review.openstack.org/#/c/442804/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/442804/</a> [2]<br>
> <a href="https://review.openstack.org/#/c/431691/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/431691/</a> [3]<br>
> <a href="https://review.openstack.org/#/q/topic:fix-os-log-merger-crash" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/q/topic:fix-os-log-merger-<wbr>crash</a><br>
><br>
> Thanks,<br>
> Ihar<br>
<br>
</div></div>Hi Ihar,<br>
<br>
That is an unexpected case that never came up during our tests or usage,<br>
but it is indeed something the script should take into account.<br>
<br>
Thanks for the os-log-merger patches, I've reviewed them and they look<br>
good to me, so hopefully they'll land before you come back online.  ;-)<br>
<br>
Cheers,<br>
Gorka.<br>
</blockquote></div><br></div>