<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 7, 2016 at 3:22 PM, Gianluca Cecchi <span dir="ltr"><<a href="mailto:gianluca.cecchi@gmail.com" target="_blank">gianluca.cecchi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><span class="gmail-"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 7, 2016 at 2:59 PM, Nir Soffer <span dir="ltr"><<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid 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:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><span><div><br></div></span>wasn’t it suppose to be fixed to reuse the connection? Like all the other clients (vdsm migration code:-) </div></div></blockquote><div><br></div></span><div>This is orthogonal issue.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div>Does schema validation matter then if there would be only one connection at the start up?</div></div></blockquote><div><br></div></span><div>Loading once does not help command line tools like vdsClient, hosted-engine and</div><div>vdsm-tool. </div><span><font color="#888888"><div><br></div><div>Nir</div></font></span><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><br><blockquote type="cite"><div><span><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid 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:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div><br></div></span><div>Simone, reusing the connection is good idea anyway, but what you describe is </div><div>a bug in the client library. The library does *not* need to load and parse the</div><div>schema at all for sending requests to vdsm.</div><div><br></div><div>The schema is only needed if you want to verify request parameters,</div><div>or provide online help, these are not needed in a client library.</div><div><br></div><div>Please file an infra bug about it.</div></div></div></div></blockquote><div><br></div></span><div>Done, <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1381899" target="_blank">https://bugzilla.redhat.com/sh<wbr>ow_bug.cgi?id=1381899</a></div></div></div></div></blockquote><div><br></div><div>Here is a patch that should eliminate most most of the problem:</div><div><a href="https://gerrit.ovirt.org/65230" target="_blank">https://gerrit.ovirt.org/65230</a><br></div><div><br></div><div>Would be nice if it can be tested on the system showing this problem.</div><div><br></div><div>Cheers,</div><div>Nir</div></div></div></div></span><span>
______________________________<wbr>_________________<br></span></div></blockquote></div></div></blockquote></span></div></div></div></blockquote></div><br></div><div class="gmail_extra"><br></div></span><div class="gmail_extra">this is a video of 1 minute with the same system as the first post, but in 4.0.3 now and the same 3 VMs powered on without any particular load.</div><div class="gmail_extra">It seems very similar to the previous 3.6.6 in cpu used by ovirt-ha-agent.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><a href="https://drive.google.com/file/d/0BwoPbcrMv8mvSjFDUERzV1owTG8/view?usp=sharing" target="_blank">https://drive.google.com/file/<wbr>d/<wbr>0BwoPbcrMv8mvSjFDUERzV1owTG8/<wbr>view?usp=sharing</a><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Enjoy Nir ;-)<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">If I can apply the patch also to 4.0.3 I'm going to see if there is then a different behavior. </div><div class="gmail_extra">Let me know,</div><div class="gmail_extra"><br></div></div></blockquote><div><br></div><div>I'm trying it right now.</div><div>Any other tests will be really appreciated.</div><div><br></div><div>The patch is pretty simply, you can apply that on the fly.</div><div>You have to shutdown ovirt-ha-broker and ovirt-ha-agent; then you could directly edit </div><div>/usr/lib/python2.7/site-packages/api/vdsmapi.py<br></div><div>around line 97 changing from</div><div>loaded_schema = yaml.load(f)<br></div><div>to</div><div>loaded_schema = yaml.load(f, Loader=yaml.CLoader)<br></div><div>Please pay attention to keep exactly the same amount of initial spaces.<br></div><div><br></div><div>Then you can simply restart the HA agent and check.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"></div><div class="gmail_extra">Gianluca</div></div>
<br>______________________________<wbr>_________________<br>
Users mailing list<br>
<a href="mailto:Users@ovirt.org">Users@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/users" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/users</a><br>
<br></blockquote></div><br></div></div>