<div dir="ltr">Hi Christophe,<br><br>I'll do some testing this evening and will let you know.<br><br>Thanks!<br>iordan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Dec 4, 2013 at 10:30 AM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey,<br>
<div class="im"><br>
On Tue, Dec 03, 2013 at 06:21:22PM +0100, Christophe Fergeau wrote:<br>
> On Tue, Dec 03, 2013 at 11:46:40AM -0500, i iordanov wrote:<br>
> > Would you like me to generate a patch with all the suggested changes and a<br>
> > better log entry?<br>
><br>
> I've been doing that actually, with various changes:<br>
> - renamed allocatevm to allocate_vm<br>
> - renamed _vmpool_ to _vm_pool<br>
> - moved ovirt_vm_pool_xml.c to ovirt_vm_pool.c as it was not that big<br>
> - used guint for the various VmPool properties<br>
> - detect parsing errors when parsing numbers<br>
> - fixed the copyright headers (the new files should be (C) 2013 yourself,<br>
>   not (C) 2013 Red Hat Inc)<br>
><br>
> I still have these changes roughly split up if you don't like some of them<br>
> and want me to drop them.<br>
<br>
</div>I've also added an async active_vm variant, and pushed all of this to<br>
<a href="http://cgit.freedesktop.org/~teuf/govirt/log/" target="_blank">http://cgit.freedesktop.org/~teuf/govirt/log/</a><br>
It would be nice if you could check I did not break anything for you (I<br>
haven't tested ovirt_vm_pool_activate_vm() at all). Then I can push all of<br>
it to git master ;)<br>
<br>
Thanks,<br>
<br>
Christophe<br>
</blockquote></div><br><br clear="all"><br>-- <br>The conscious mind has only one thread of execution.
</div>