<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 30, 2017 at 11:28 AM, Martin Sivak <span dir="ltr"><<a href="mailto:msivak@redhat.com" target="_blank">msivak@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 dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div>I think a reasonable alternative is to revert the change, adjust HE to use both formats, and then introduce the new format. The first step (revert) should be done for 4.2.0 (today?) and the rest for 4.2.1.<br></div></span><span class="m_7072077848926355922HOEnZb"><font color="#888888"><div>Y.</div></font></span><span><div><br></div></span></div></div></div></blockquote><div><br></div></span><div>The format is still the same, we just needed to autodetect the used NS and then query for attributes using the detected namespace. Patch posted, tests are passing, but since we want to be 100% sure we will test with OST and Node 0 before pushing it. Should not take long.</div></div></div></div></blockquote><div><br></div><div><br></div><div>Verified and Merged.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="HOEnZb"><font color="#888888"><div>Martin</div></font></span><span class=""><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><table border="0" cellspacing="0"><tbody><tr>
</tr>
<tr>
<td align="right" height="17"><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1516113" target="_blank">1516113</a></td>
<td align="left">cockpit-ovirt</td>
<td align="left"><a href="mailto:phbailey@redhat.com" target="_blank">phbailey@redhat.com</a></td>
<td align="left">POST</td>
<td align="left">Deploy the HostedEngine failed with the default CPU type</td></tr></tbody></table></div></div></blockquote><div><br></div></span><div>Would be happy if the remaining patch could get reviewed quickly.</div></div></div></div></blockquote><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><table border="0" cellspacing="0"><tbody><tr>
</tr>
<tr>
<td align="right" height="17"><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1518693" target="_blank">1518693</a></td>
<td align="left">ovirt-engine</td>
<td align="left"><a href="mailto:akrejcir@redhat.com" target="_blank">akrejcir@redhat.com</a></td>
<td align="left">POST</td>
<td align="left">Quota is needed to copy template disk</td></tr></tbody></table></div></div></blockquote><div><br></div></span><div>This is only via REST and the default quota can be used as a workaround - why is this a blocker?</div></div></div></div></blockquote><div><br></div><div>Automation added it because Raz marked it as Regression. But the change was intentional.</div><div> </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><table border="0" cellspacing="0"><tbody><tr>
</tr>
<tr>
<td align="right" height="17"><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1517810" target="_blank">1517810</a></td>
<td align="left">ovirt-engine</td>
<td align="left"><a href="mailto:stirabos@redhat.com" target="_blank">stirabos@redhat.com</a></td>
<td align="left">Adding additional ha-host fails.</td></tr></tbody></table></div></div></blockquote></span></div></div></div></blockquote><div><br></div><div>This one has a verified engine fix now, we just need to merge it and update Node 0 setup (also verified).</div><span class="m_7072077848926355922m_-4876262491660092973m_7051580001592472347gmail-HOEnZb"><font color="#888888"><div><br></div><div><br></div><div>Martin</div></font></span></div><br></div></div>
<br></span><span>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br></span></blockquote></div><br></div></div>
</blockquote></span></div><br></div></div>
</blockquote></span></div><br></div></div>
</blockquote></div><br></div></div>