Is the CSIT patch to go in? There are comments at the end of the patch that I thought needed to be addressed.
On Mon, Jan 28, 2019 at 1:53 AM Dimple Jain <dimple.j...@ericsson.com> wrote: > Hi Sam, > > Both the patches had to be merged together. > > As my review has been merged but genius CSIT patch is not; Genius CSIT > jobs are failing. > > > > Thanks & Regards, > > Dimple. > > > > > > *From:* Vishal Thapar [mailto:vtha...@redhat.com] > *Sent:* Monday, January 28, 2019 7:12 AM > *To:* Sam Hague <sha...@redhat.com> > *Cc:* Dimple Jain <dimple.j...@ericsson.com>; odl netvirt dev < > netvirt-...@lists.opendaylight.org>; 'genius-...@lists.opendaylight.org' ( > genius-...@lists.opendaylight.org) <genius-...@lists.opendaylight.org>; > sfc-dev@lists.opendaylight.org; Brady Johnson <bjohn...@inocybe.ca> > *Subject:* Re: [genius-dev] [netvirt-dev] [sfc-dev] Disable BFD > monitoring for tunnels by default > > > > > > > > On Mon, Jan 28, 2019 at 4:21 AM Sam Hague <sha...@redhat.com> wrote: > > > > On Wed, Jan 23, 2019, 5:58 AM Dimple Jain <dimple.j...@ericsson.com wrote: > > Hi Sam, > > I have raised a patch to disable BFD by default and Karthika has modified > genius csit accordingly. > > Here are the patches for the same. Both the patches should be merged > together. > > Can you please review and merge. > > The genius patch is merged. Does it matter if the CSIT patch is not? > > It will need to be merged. Genius CSIT's BFD suite has tests for default > behavior which will now change. > > > > And do we need anything for netvirt? There is a direct tunnel suite there > also. > > Nope. Direct tunnels are a different change, will likely be done during > Sodium. > > > > And what about the plain netvirt CSIT, is that just going to have bfd > disabled? > > Yes. Code in netvirt never really used BFD state changes. We also plan to > move to OFTunnels soon and BFD is not supported with OFTunnels. > > > > https://git.opendaylight.org/gerrit/#/c/79629/ > > https://git.opendaylight.org/gerrit/#/c/79836/ > > > > > > Thanks & Regards, > > Dimple. > > > > > > *From:* netvirt-dev-boun...@lists.opendaylight.org [mailto: > netvirt-dev-boun...@lists.opendaylight.org] *On Behalf Of *Brady Johnson > *Sent:* Thursday, December 20, 2018 2:34 PM > *To:* Faseela K <faseel...@ericsson.com> > *Cc:* odl netvirt dev <netvirt-...@lists.opendaylight.org>; > genius-...@lists.opendaylight.org; sfc-dev@lists.opendaylight.org > *Subject:* Re: [netvirt-dev] [sfc-dev] Disable BFD monitoring for tunnels > by default > > > > No objections from me, thanks. > > > Regards, > > > > *Brady Johnson* > > bjohn...@inocybe.ca > > > > > > > > > > > > On Thu, Dec 20, 2018 at 6:31 AM Faseela K <faseel...@ericsson.com> wrote: > > Brady/David, > > Any objections from sfc side? > > Thanks, > > Faseela > > > > *From:* Sam Hague [mailto:sha...@redhat.com] > *Sent:* Wednesday, December 19, 2018 10:30 PM > *To:* Faseela K <faseel...@ericsson.com> > *Cc:* genius-...@lists.opendaylight.org; sfc-dev@lists.opendaylight.org; > odl netvirt dev <netvirt-...@lists.opendaylight.org>; Brady Allen Johnson > <bjohn...@inocybe.ca> > *Subject:* Re: [sfc-dev] Disable BFD monitoring for tunnels by default > > > > > > > > On Sat, Dec 15, 2018 at 9:42 AM Faseela K <faseel...@ericsson.com> wrote: > > Hi, > > > > There was a discussion in last genius weekly call, whether we can > disable tunnel monitoring by default. > > Vishal indicated that netvirt would prefer disabling monitoring by > default. > > And can be enabled explicitly if required. > > Any concerns if we do that? > > This is a good idea. We talked about this a while back also and thought we > decided to do the same. The upstream netvirt csit does not use bfd. > > > > Does disabling the bfd have any impact on the tunnel issues that are being > debugged or is this a benign change? > > > > This would clean up the logs also as the bfd stuff is noisy in the > ovsdb/vswitchd logs that the upstream csit collects. > > > > Thanks, > > Faseela > > _______________________________________________ > sfc-dev mailing list > sfc-dev@lists.opendaylight.org > https://lists.opendaylight.org/mailman/listinfo/sfc-dev > > _______________________________________________ > genius-dev mailing list > genius-...@lists.opendaylight.org > https://lists.opendaylight.org/mailman/listinfo/genius-dev > >
_______________________________________________ sfc-dev mailing list sfc-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/sfc-dev