Correct, we will only change plugin in master, so we will only consider: https://git.opendaylight.org/gerrit/#/c/33303/ <https://git.opendaylight.org/gerrit/#/c/33303/>
Also Sai, do you know why this patch says it cannot be merged? BR/Luis > On Apr 20, 2016, at 4:59 PM, Abhijit Kumbhare <[email protected]> wrote: > > Hi Sai, > > Please don't change the default on the stable/beryllium to the Li version. We > are only changing it to default in Boron. > > Thanks, > Abhijit > > On Wednesday, April 20, 2016, Sai MarapaReddy <[email protected] > <mailto:[email protected]>> wrote: > Hi Luis, > > Following are the patches for Stable/beryllium & master for L2switch. I have > change feature.xml & pom.xml to correctly point the li version. > > Stable/beryllium - https://git.opendaylight.org/gerrit/#/c/33304/ > <https://git.opendaylight.org/gerrit/#/c/33304/> > Master - https://git.opendaylight.org/gerrit/#/c/33303/ > <https://git.opendaylight.org/gerrit/#/c/33303/> > > Could you please give it one more try with these patches ? > > Regards > Sai MarapaReddy > Brocade Communications > > On Fri, Apr 15, 2016 at 3:57 PM, Luis Gomez <[email protected] > <javascript:_e(%7B%7D,'cvml','[email protected]');>> wrote: > I think the table is more like this: > > Case 1: Current L2 switch (should point to OFP-He) & Current OFP (default > plugin is OFP-He) --> Should pass, current situation > Case 2: L2 switch with patch (should point to OFP-Li) & Current OFP (default > plugin is OFP-He) --> Should pass if current L2switch patch contained all > fixes for Li plugin. > Case 3: Current L2 switch (should point to OFP-He) & OFP with patch > (default plugin is OFP-Li, no -li features) --> Should pass if L2switch did > not care about OF plugin version > Case 4: L2 switch with patch (should point to OFP-Li) & OFP with patch > (default plugin is OFP-Li, no -li features) --> This will never work, you > are pulling -li features which do not exist anymore in OFP. > > I tested 2 & 3. > >> On Apr 15, 2016, at 2:42 PM, Sai MarapaReddy <[email protected] >> <javascript:_e(%7B%7D,'cvml','[email protected]');>> wrote: >> >> Hi Luis, >> >> Please correct me if i am wrong here. >> >> You have checked below Case 3 & Case 4 and found errors. >> >> Case 3 errors :- >> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/291/robot/report/log.html >> >> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/291/robot/report/log.html> >> Case 4 erros :- >> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/296/robot/report/log.html >> >> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/296/robot/report/log.html> >> >> L2Switch Patch - https://git.opendaylight.org/gerrit/#/c/33304/ >> <https://git.opendaylight.org/gerrit/#/c/33304/> & >> https://git.opendaylight.org/gerrit/#/c/33303/ >> <https://git.opendaylight.org/gerrit/#/c/33303/> >> >> OFP Patch - https://git.opendaylight.org/gerrit/#/c/35892/ >> <https://git.opendaylight.org/gerrit/#/c/35892/> >> >> Current L2 switch code (with out patch ) points to features-openflowplugin >> which is equivalent to OFP-he >> >> Case 1: Current L2 switch (should point to OFP-He) & Current OFP (should >> point to OFP-He) --> Should pass >> Case 2: L2 switch with patch (should point to OFP-Li) & Current OFP (should >> point to OFP-He) --> Should Fail >> Case 3: Current L2 switch (should point to OFP-He) & OFP with patch >> (should point to OFP-Li) --> Should Fail >> Case 4: L2 switch with patch (should point to OFP-Li) & OFP with patch >> (should point to OFP-Li) --> Should Pass >> >> >> >> Regards, >> Sai Reddy >> Brocade Communications >> >> On Thu, Apr 14, 2016 at 4:03 PM, Luis Gomez <[email protected] >> <javascript:_e(%7B%7D,'cvml','[email protected]');>> wrote: >> And this second test report is when I just apply ofplugin Li patch [2]: >> >> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/296/robot/report/log.html >> >> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/296/robot/report/log.html> >> >> I see SAME issues as with l2switch Li plugin patch + some weird topology >> display issue (internal topology works as otherwise ping would not work but >> switch links are not shown) the second time we run Host Tracker test. >> >> BR/Luis >> >> [2] https://git.opendaylight.org/gerrit/#/c/35892/ >> <https://git.opendaylight.org/gerrit/#/c/35892/> >> >> >>> On Apr 13, 2016, at 7:32 PM, Luis Gomez <[email protected] >>> <javascript:_e(%7B%7D,'cvml','[email protected]');>> wrote: >>> >>> From today’s l2switch call, I got action point to run system test on Li >>> patch [1]: >>> >>> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/291/robot/report/log.html >>> >>> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/291/robot/report/log.html> >>> >>> From the result above we can observe following issues: >>> >>> - Flow push: There is no LLDP flows being pushed. Also flood flows seem to >>> be in config space because operational shows flow ID. >>> - Address Tracker: All host IP addresses are seeing in all nodes, that is >>> not right, only node connected to host should show the host. >>> - Host Tracker: After remove/add OpenFlow port (ovs-vsctl add-port), no >>> host is learned anymore through that port. >>> - Loop Remover: Not working, using a loop topology, even when we observe >>> some ports are in DISCARD mode, mininet sees DUP packets. >>> >>> BR/Luis >>> >>> [1] https://git.opendaylight.org/gerrit/#/c/33303/ >>> <https://git.opendaylight.org/gerrit/#/c/33303/> >>> >>> >> >> >> _______________________________________________ >> L2switch-dev mailing list >> [email protected] >> <javascript:_e(%7B%7D,'cvml','[email protected]');> >> https://lists.opendaylight.org/mailman/listinfo/l2switch-dev >> <https://lists.opendaylight.org/mailman/listinfo/l2switch-dev> >> >> > >
_______________________________________________ L2switch-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/l2switch-dev
