<div dir="ltr">Cool!<div><br></div><div>This needs integration tests with real vdsm, or at least a server using vdsm</div><div>yajsonrpc code. I'm worried about incompatibilities between the go stomp</div><div>library and our own stomp implementation, not used by any other code.</div><div><br></div><div>When it works, we can convert vdsm-client to go :-)</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jul 15, 2017 at 8:53 AM Yaniv Bronheim <<a href="mailto:ybronhei@redhat.com">ybronhei@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Great! make it official under ovirt imo . This will be totally useful later on with openshift integration. Im almost convinced that once ovirt will run in parallel to openshift or as part of openshift, we'll need to call vdsm api commands via modules that with high chance will be written in go. Give specific example won't be meaningful much because we still designing all this vm+containers architecture and flows.<div>Thanks</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 14, 2017 at 4:40 PM Adam Litke <<a href="mailto:alitke@redhat.com" target="_blank">alitke@redhat.com</a>> wrote:<br></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">On Fri, Jul 14, 2017 at 9:32 AM, Piotr Kliczewski <span dir="ltr"><<a href="mailto:piotr.kliczewski@gmail.com" target="_blank">piotr.kliczewski@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Fri, Jul 14, 2017 at 3:14 PM, Dan Kenigsberg <<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>> wrote:<br>
> On Fri, Jul 14, 2017 at 3:11 PM, Piotr Kliczewski<br>
> <<a href="mailto:piotr.kliczewski@gmail.com" target="_blank">piotr.kliczewski@gmail.com</a>> wrote:<br>
>> All,<br>
>><br>
>> I pushed very simple jsonrpc go client [1] which allows to talk to<br>
>> vdsm. I had a request to create it but if there are more people<br>
>> willing to use it I am happy to maintain it.<br></span></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Awesome Piotr! Thanks for the great work.</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
>><br>
>> Please let me know if you find any issues with it or you have any<br>
>> feature requests.<br>
><br>
> Interesting. Which use case do you see for this client?<br>
> Currently, Vdsm has very few clients: Engine, vdsm-client, mom and<br>
> hosted-engine. Too often we forget about the non-Engine ones and break<br>
> them, so I'd be happy to learn more about a 5th.<br>
<br>
</span>Adam asked for the client for his storage related changes. I am not<br>
sure about specific use case.<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>I am looking at implementing a vdsm flexvol driver for kubernetes. This would allow kubernetes pods to access vdsm volumes using the native PV and PVC mechanisms.</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><br>
><br>
> Regarding <a href="https://github.com/pkliczewski/vdsm-jsonrpc-go/blob/master/example/main.go" rel="noreferrer" target="_blank">https://github.com/pkliczewski/vdsm-jsonrpc-go/blob/master/example/main.go</a><br>
> : programming without exceptions and try-except is a pain. don't you<br>
> need to check the retval of Subscribe and disconnect on failure?<br>
<br>
</span>By no means example is not perfect and you are correct. I will fix.<br>
</blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><br><br clear="all"><div><br></div>-- <br><div class="m_4815198233745745703m_6180150326172258710gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Adam Litke</div></div>
</div></div>
_______________________________________________<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/listinfo/devel</a></blockquote></div><div dir="ltr">-- <br></div><div class="m_4815198233745745703gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Yaniv Bronhaim.</div></div>
_______________________________________________<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/listinfo/devel</a></blockquote></div>