<div dir="ltr"><div><div><div><div><div>Piotr,<br><br></div>According to our checks engine succeed connecting VDSM through ipv6 once the work you've reminded has been done. So it looks like jsonrpc is agnostic to IP layer it uses underneath.<br><br></div>Could you please elaborate on why do you think that we break backward compatibility.<br>If you compare the wiki to vdsmapi-schema.json file, looks like the last isn't updated for some time and needs to be updated according to the actual code.<br><br></div>We add dhcpv6 in order to allow user to configure v6 stack independently from v4 one. That could be useful especially in a transition period, when a user have fully configured ipv4 and would like to try using ipv6 and he hasn't configured his DHCP server for v6 yet.<br><br></div>Regards,<br></div>Yevgeny<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 17, 2016 at 10:26 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">Thank you for the effort.<br>
<br>
I have 2 comments:<br>
- In [1] I do not see any information about using ipv6 in jsonrpc<br>
code. I know that there was some work done on vdsm [2]<br>
side but nothing on the engine side.<br>
<br>
- We are about to break some backward compatibility so I suggest to<br>
define new type which would hold ipv4 or ipv6<br>
information (vdsm api section) or if we want to keep the backward<br>
compatibility we should reuse existing parameters<br>
and prepare the code to handle both formats.<br>
Why we are adding @dhcpv6? We do not have this setting for ipv4.<br>
<br>
Thanks,<br>
Piotr<br>
<br>
[2] <a href="https://gerrit.ovirt.org/51319" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/51319</a><br>
<div><div class="h5"><br>
<br>
On Tue, Feb 16, 2016 at 5:02 PM, Yevgeny Zaspitsky <<a href="mailto:yzaspits@redhat.com">yzaspits@redhat.com</a>> wrote:<br>
> Hi All,<br>
><br>
> IPv6 support is a long waited feature in oVirt and that was decided to<br>
> include it in v4.0.<br>
> I have updated the feature page [1] with the recent projected changes.<br>
><br>
> Please take a look on the page and please send me your comments so those<br>
> would be reflected in the feature implementation.<br>
><br>
> Regards,<br>
> Yevgeny<br>
><br>
> [1] <a href="http://www.ovirt.org/Features/IPv6_support" rel="noreferrer" target="_blank">http://www.ovirt.org/Features/IPv6_support</a><br>
><br>
</div></div>> _______________________________________________<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/mailman/listinfo/devel</a><br>
</blockquote></div><br></div>