I'm looking at the csit patch today. it isn't done yet, afaik.
JamO On 1/28/19 8:12 AM, Sam Hague wrote:
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 <[email protected] <mailto:[email protected]>> 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:[email protected] <mailto:[email protected]>] *Sent:* Monday, January 28, 2019 7:12 AM *To:* Sam Hague <[email protected] <mailto:[email protected]>> *Cc:* Dimple Jain <[email protected] <mailto:[email protected]>>; odl netvirt dev <[email protected] <mailto:[email protected]>>; '[email protected] <mailto:[email protected]>' ([email protected] <mailto:[email protected]>) <[email protected] <mailto:[email protected]>>; [email protected] <mailto:[email protected]>; Brady Johnson <[email protected] <mailto:[email protected]>> *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 <[email protected] <mailto:[email protected]>> wrote:____ __ __ On Wed, Jan 23, 2019, 5:58 AM Dimple Jain <[email protected] <mailto:[email protected]> 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:*[email protected] <mailto:[email protected]> [mailto:[email protected] <mailto:[email protected]>] *On Behalf Of *Brady Johnson *Sent:* Thursday, December 20, 2018 2:34 PM *To:* Faseela K <[email protected] <mailto:[email protected]>> *Cc:* odl netvirt dev <[email protected] <mailto:[email protected]>>; [email protected] <mailto:[email protected]>; [email protected] <mailto:[email protected]> *Subject:* Re: [netvirt-dev] [sfc-dev] Disable BFD monitoring for tunnels by default____ ____ No objections from me, thanks.____ ____ Regards,____ ____ *Brady Johnson*____ [email protected] <mailto:[email protected]>____ ____ ____ ____ ____ ____ ____ On Thu, Dec 20, 2018 at 6:31 AM Faseela K <[email protected] <mailto:[email protected]>> wrote:____ Brady/David,____ Any objections from sfc side?____ Thanks,____ Faseela____ ____ *From:*Sam Hague [mailto:[email protected] <mailto:[email protected]>] *Sent:* Wednesday, December 19, 2018 10:30 PM *To:* Faseela K <[email protected] <mailto:[email protected]>> *Cc:* [email protected] <mailto:[email protected]>; [email protected] <mailto:[email protected]>; odl netvirt dev <[email protected] <mailto:[email protected]>>; Brady Allen Johnson <[email protected] <mailto:[email protected]>> *Subject:* Re: [sfc-dev] Disable BFD monitoring for tunnels by default____ ____ ____ ____ On Sat, Dec 15, 2018 at 9:42 AM Faseela K <[email protected] <mailto:[email protected]>> 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 [email protected] <mailto:[email protected]> https://lists.opendaylight.org/mailman/listinfo/sfc-dev____ _______________________________________________ genius-dev mailing list [email protected] <mailto:[email protected]> https://lists.opendaylight.org/mailman/listinfo/genius-dev____ _______________________________________________ netvirt-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/netvirt-dev
_______________________________________________ sfc-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/sfc-dev
