PSB inline. On Thu, May 25, 2017 at 5:59 PM, N Vivekanandan <[email protected] > wrote:
> Hi Sam/Sridhar, > > For tempest IPV6 tests to work, is the IPv6 PING responder the only > pending requirement. > Over the last few months, we were evaluating what it takes to enable tempest IPv6 scenario tests along with @Alon and his team. AFAIK, almost all the necessary changes were addressed in ODL netvirt. The only pending thing in ODL/Netvirt is the support for ping6. Once we have this feature in, we can enable tempest scenario tests for IPv6. > Or we want to start trying Tempest IPv6 in Nitrogen and this would be the > first fix. > Nope, this is not the case. > > > If it’s the latter, would be good to follow the Spec process. > > If it’s the former, +1 from me. > > > Thanks @Vivek. > -- > > Thanks, > > > > Vivek > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *Sam Hague > *Sent:* Thursday, May 25, 2017 5:47 PM > *To:* Sridhar Gaddam <[email protected]> > *Cc:* odl netvirt dev <[email protected]>; > [email protected]; openflowplugin-dev@lists. > opendaylight.org > *Subject:* Re: [netvirt-dev] [genius-dev] [openflowplugin-dev] Review > request - Support for ping6 in ODL > > > > I don't think we need a spec as some functionality is broken now. The > table 19 patch broke things so we need this fixed and we need tempest > enabled to mitigate breaking it in the future. I could also see this > following under the previous ipv6 specs as a minor overlooked item. > > > > On Thu, May 25, 2017 at 8:01 AM, Sridhar Gaddam <[email protected]> > wrote: > > Hello Vishal, > > > > Its a single flow (which is supported on OVS2.5) similar to IPv4 ping and > impacts only IPv6. There will not be any change/impact to IPv4 pipeline. > > Currently we are not running IPv6 tempest scenario tests because of this. > > > > As you know, we are adding new features in Netvirt and this is breaking > IPv6 use-cases (fe., [1]). Sometimes we are not even aware that some > existing IPv6 use-cases are broken. > > So, enabling tempest IPv6 scenario tests would help us to make sure that > new changes are not creating issues to the existing IPv6 features. > > IMHO I don't feel that a spec is required for this use-case and would > appreciate if we can get this in as soon as possible. > > > > [1] https://git.opendaylight.org/gerrit/#/c/57719/2 > > > > Thanks, > > --Sridhar. > > > > On Thu, May 25, 2017 at 5:19 PM, Vishal Thapar <[email protected]> > wrote: > > Hi Sridhar, > > > > I’d recommend raising a genius/netvirt spec for this. ICMPv6 responder is > a new feature and impacts pipeline so should go proper design review et al. > > > > Regards, > > Vishal. > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *Sridhar > Gaddam > *Sent:* 25 May 2017 16:44 > *To:* [email protected]; openflowplugin-dev@lists. > opendaylight.org; odl netvirt dev <[email protected]> > *Subject:* [openflowplugin-dev] Review request - Support for ping6 in ODL > > > > Hello Genius/OpenflowPlugin teams, > > > > In order to enable tempest IPv6 scenario tests with ODL/Netvirt, ping6 > support for the Neutron router interfaces has to be supported. > > This is currently a missing gap, so we could not enable tempest IPv6 > scenario tests so far. > > > > I was looking at this use-case and figured out that we can support ping6 > similar to ping (in IPv4) by programming the necessary OVS flows that > auto-respond to ping6 requests from the VMs. > > > > Following are the five dependent patches that are required to be merged in > GENIUS and OpenflowPlugin for this use-case. > > > > I would appreciate if Openflowplugin and GENIUS teams take a look at the > following dependent patches so that I can propose the necessary changes in > Netvirt. > > > > *OpenflowPlugin patches:* > > 1. Add Nicira extension support for matching IPv6 Src/Dst - > https://git.opendaylight.org/gerrit/#/c/56969/ > > 2. Bug 8535: Fix IPv6 OXMHeader Mask issue - https://git.opendaylight.org/ > gerrit/#/c/57694/ > > > > *GENIUS patches:* > > 1. Utility api to configure icmpv6 type - https://git.opendaylight.org/ > gerrit/#/c/57072/ > > 2. Support for moving Src/Dest IPv6 addresses in NxRegMove action - > https://git.opendaylight.org/gerrit/#/c/57027/ > > 3. Support API that configures IPv6 address as sourceAddress - > https://git.opendaylight.org/gerrit/#/c/57026/ > > > > Thank you, > > --Sridhar. > > > > > > > > > _______________________________________________ > genius-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/genius-dev > > >
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
