<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 4, 2017 at 12:29 PM, Roy Golan <span dir="ltr"><<a href="mailto:rgolan@redhat.com" target="_blank">rgolan@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>I'm working on a POC lately on a change to stats collection and retrieval by VDSM. The moto is to cut all we can from host/vm stats (possibly caps) and report only core-business stuff to the engine. Engine will retrieve the rest through a 3rd party provider (nevermind what is it atm)<br><br></div><div>Being backward compatible by design, I have to support 2 API versions for Host.getStats , '4.1' and '4.2'.<br></div><div>Except from supplying less parameters, I want VDSM to do less stuff. It doesn't need to sample what it doesn't report. In other words I want '4.1-sampling' and '4.2-sampling'<br><br></div><div># Introducing 'configuration' Verb:<br><br></div><div>As engine knows always(Hosted Engine as well) what cluster version this host belongs to, it can configure VDSM to operate in cluster version mode.<br></div><div><br> Host.configure(config={<wbr>version: 4.2}<br></div><div><br></div><div>Consider this verb, pre-activating using 'Host.getCaps' to set the context.<br>It will set the righjt sampling method, and other stuff if needed then API endpoints will have the right permutation of the api to answer it.<br><br></div><div>4.2 host can operate in 4.1 mode:<br> Host.configure(config={<wbr>version: 4.1}<br><br></div><div>Issue: moving a 4.2 host from 4.2 cluster to 4.1 is a problem since engine needs to know this is a new vdsm that has the verb available. One way to overcome that is to fire the verb for every host regardless of the version and disregard an error that implies the verb doesn't exist.<br></div></div></blockquote><div><br></div><div>Isn't it solved by host re-installation?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div># Engine:<br></div><div>Engine will have a handling of the verb per version.<br></div><div>Host/Vms monitoring should be changed - I suggest to move out of the monitoring code the whole stats collection as it is a different task which is orthogonal to 'monitoring' and in 4.2 more than before.<br><br><br></div><div>I know configuration for VDSM has been discussed before and there are probably tons of ways to do it. When you share your thoughts please remember that configuration is a by-product of the effort.<br></div></div></blockquote><div><br></div><div>How do we persist this level on VDSM? Or we don't, and if VDSM is restarted it is again back to 4.1 mode until Engine tells it otherwise?</div><div>Y.</div><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>Nevertheless it can be potentially beneficial to more functions in vdsm.<br></div><div><br></div><div>Thanks,<br></div><div>Roy<br></div><div><br></div></div>
<br>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br></div></div>