On 10/19/2016 11:26 PM, Sai MarapaReddy wrote: > Address tracker tries [1] to read any existing address from data store. My > guess is in the case of immediate restart of > mininet , addresses from data store are not cleaned and this is why we see > all addresses. > > In further attempt to verify this information I have a patch here [2]. I > tried running it with keyword "test-l2switch-all" > but no success. Could you please run it. This might help in troubleshooting > the issue further.
it's because we don't have a patch test job for l2switch yet. It's something projects are adding on an as-needed basis. I gave you one here: https://git.opendaylight.org/gerrit/#/c/47219/ give it a +1 and we can try to get it merged. Thanks, JamO > [1] > https://github.com/opendaylight/l2switch/blob/stable/boron/addresstracker/implementation/src/main/java/org/opendaylight/l2switch/addresstracker/addressobserver/AddressObservationWriter.java#L116 > > [2] https://git.opendaylight.org/gerrit/#/c/47186/ > > On Wed, Oct 19, 2016 at 9:43 AM, Luis Gomez <ece...@gmail.com > <mailto:ece...@gmail.com>> wrote: > > Hi Sai, after looking at the test more in detail this is not exactly the > behavior (sorry for the confusion), what is > really happening is: > > - Mininet restarts > - There is no host in address-tracker after restart > - We do the classic mininet pingall test > - After the pingall test the host addresses (10.0.0.1, 10.0.0.2, > 10.0.0.3) are seen in all switches, while in normal > scenario 10.0.0.1 should be only in s1 to-host port, 10.0.0.2 should be > only in s2 to-host port and 10.0.0.3 should be > only in s3 to-host port. > > The normal scenario happens when we leave 2 secs between mininet stop + > start. > > So 2 questions: > > - What is the impact of address tracker registering remote IP addresses > in the switch-to-switch ports? As far as I can > see flows are generated correctly even in this case. > - Any idea why the address tracker would get confused and add IP > addresses to the switch-to-switch ports? maybe the > application fails to identify these ports as going switch-to-switch? > > BR/Luis > > >> On Oct 19, 2016, at 7:42 AM, Luis Gomez <ece...@gmail.com >> <mailto:ece...@gmail.com>> wrote: >> >> Timer=600 sec but in Boron when we leave just 2 secs between mininet >> stop & start, we do not see any IP address, also >> in Beryllium we do no see any IP address after restarting mininet, so >> are you sure switch disconnect/connect does not >> impact address tracker cache? >> >> BR/Luis >> >>> On Oct 19, 2016, at 3:37 AM, Sai MarapaReddy <sai.marapare...@gmail.com >>> <mailto:sai.marapare...@gmail.com>> wrote: >>> >>> Hi Luis, >>> >>> Addresses will be removed if this condition [1] passes. >>> >>> Default value of time 'timeStampUpdateInterval' is mentioned here [2]. >>> >>> Please try reducing the default value of time to make sure to addresses >>> are flushed. >>> >>> Since every time the mininet connects with same IP / Mac values, >>> addresses will only be removed if condition at [1] is >>> met. >>> >>> >>> >>> [1] >>> https://github.com/opendaylight/l2switch/blob/master/addresstracker/implementation/src/main/java/org/opendaylight/l2switch/addresstracker/addressobserver/AddressObservationWriter.java#L144 >>> >>> <https://github.com/opendaylight/l2switch/blob/master/addresstracker/implementation/src/main/java/org/opendaylight/l2switch/addresstracker/addressobserver/AddressObservationWriter.java#L144> >>> >>> [2] >>> https://github.com/opendaylight/l2switch/blob/master/addresstracker/implementation/src/main/yang/address-tracker-config.yang#L17 >>> >>> <https://github.com/opendaylight/l2switch/blob/master/addresstracker/implementation/src/main/yang/address-tracker-config.yang#L17> >>> >>> >>> >>> On Tue, Oct 18, 2016 at 7:30 PM, Luis Gomez <ece...@gmail.com >>> <mailto:ece...@gmail.com>> wrote: >>> >>> Thanks Sai, yes I have a question regrading the first issue below: >>> any idea why address tracker would keep >>> discovered IP address information after restarting mininet? like >>> what is today in the code to flush the address >>> cache and why this is not working when mininet restarts? >>> >>> BR/Luis >>> >>>> On Oct 18, 2016, at 6:56 PM, Sai MarapaReddy >>>> <sai.marapare...@gmail.com <mailto:sai.marapare...@gmail.com>> wrote: >>>> >>>> Thanks Luis. >>>> >>>> Yes that is drop-all flow. Please let us know if you need any >>>> further help from l2switch. >>>> >>>> On Tue, Oct 18, 2016 at 1:03 PM, Luis Gomez <ece...@gmail.com >>>> <mailto:ece...@gmail.com>> wrote: >>>> >>>> Actually this is the drop flow so it is expected, anyway going >>>> deeper into l2switch test issues I see: >>>> >>>> - The address tracker issue is because hosts addresses are not >>>> getting cleared when we restart mininet with >>>> no sleep in-between. >>>> - The loop remover issue could be related to some test issue, >>>> I am trying to clean up/repair the test here: >>>> https://git.opendaylight.org/gerrit/#/c/47095/ >>>> <https://git.opendaylight.org/gerrit/#/c/47095/> >>>> >>>> BR/Luis >>>> >>>> > On Oct 18, 2016, at 10:02 AM, Luis Gomez <ece...@gmail.com >>>> <mailto:ece...@gmail.com>> wrote: >>>> > >>>> > There are definitely weird things going on in l2switch, the >>>> first ERROR you mention produces some weird >>>> flow (no match and no action) in the switch: >>>> > >>>> > Flow 55: >>>> > "cookie": 3098476543630901303, >>>> > "flags": "", >>>> > "hard-timeout": 0, >>>> > "id": "55", >>>> > "idle-timeout": 0, >>>> > "match": {}, >>>> > >>>> "opendaylight-flow-statistics:flow-statistics": { >>>> > "byte-count": 0, >>>> > "duration": { >>>> > "nanosecond": 239000000, >>>> > "second": 36 >>>> > }, >>>> > "packet-count": 0 >>>> > }, >>>> > "priority": 0, >>>> > "table_id": 0 >>>> > >>>> > >>>> > The second issue, I am not sure what is the impact but >>>> definitely something to look at. >>>> > >>>> > I think in general we will need l2switch support to >>>> troubleshoot and progress in current issues. >>>> > >>>> > BR/Luis >>>> > >>>> > >>>> >> On Oct 17, 2016, at 8:09 AM, Miroslav Macko >>>> <miroslav.ma...@pantheon.tech >>>> <mailto:miroslav.ma...@pantheon.tech>> wrote: >>>> >> >>>> >> Hello Luis and dev guys, >>>> >> >>>> >> There is this info from l2-switch in the karaf log: >>>> >> >>>> >> 2016-10-14 22:43:24,941 | INFO | pool-16-thread-1 | >>>> FlowWriterServiceImpl | 229 - >>>> org.opendaylight.l2switch.main.impl - 0.5.0.SNAPSHOT | In >>>> addMacToMacFlowsUsingShortestPath: No flows added. >>>> Source and Destination ports are same. >>>> >> >>>> >> Is it ok? >>>> >> >>>> >> And this debug message(it has incorrect severity): >>>> >> >>>> >> 2016-10-14 22:43:55,282 | ERROR | entLoopGroup-5-3 | >>>> DeviceFlowRegistryImpl | 210 - >>>> org.opendaylight.openflowplugin.impl - 0.4.0.SNAPSHOT | Flow >>>> with flowId 85 already exists in table 0 >>>> >> >>>> >> So it looks like that flows with the same ID are added. >>>> >> >>>> >> What in short these failing tests are doing and what it >>>> should test please? >>>> >> >>>> >> Thanks, >>>> >> Miro >>>> >> >>>> >> ________________________________________ >>>> >> Od: Luis Gomez <ece...@gmail.com <mailto:ece...@gmail.com>> >>>> >> Odoslané: 15. októbra 2016 1:01 >>>> >> Komu: Tomáš Slušný >>>> >> Kópia: An Ho; openflowplugin-dev; OpenDayLight-L2switch-Dev >>>> >> Predmet: Re: [openflowplugin-dev] [L2switch-dev] Blocker >>>> bug analysis >>>> >> >>>> >> No luck :), still fails on "address tracker" and "loop >>>> remover": >>>> >> >>>> >> >>>> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-carbon/154/ >>>> >>>> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-carbon/154/> >>>> >> >>>> >>>> https://logs.opendaylight.org/releng/jenkins092/l2switch-csit-1node-switch-only-carbon/154/archives/log.html.gz >>>> >>>> <https://logs.opendaylight.org/releng/jenkins092/l2switch-csit-1node-switch-only-carbon/154/archives/log.html.gz> >>>> >> >>>> >> BR/Luis >>>> >> >>>> >> >>>> >>> On Oct 14, 2016, at 2:26 PM, Luis Gomez <ece...@gmail.com >>>> <mailto:ece...@gmail.com>> wrote: >>>> >>> >>>> >>> OK Tomas, I will try your patch >>>> https://git.opendaylight.org/gerrit/#/c/46390/ >>>> <https://git.opendaylight.org/gerrit/#/c/46390/> on >>>> l2switch-csit-1node-switch-only-carbon and let you know. >>>> >>> >>>> >>> >>>> >>>> On Oct 14, 2016, at 1:23 PM, Tomáš Slušný >>>> <tomas.slu...@pantheon.tech >>>> <mailto:tomas.slu...@pantheon.tech>> wrote: >>>> >>>> >>>> >>>> Also, 6710 was already merged only in master branch, but >>>> not yet in boron. It have cherry-pick ready >>>> here: https://git.opendaylight.org/gerrit/#/c/46761/ >>>> <https://git.opendaylight.org/gerrit/#/c/46761/>, but >>>> until it is merged, I think it would be better to check for >>>> improvement in Jenkins test for Carbon here: >>>> >>>> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-carbon/ >>>> >>>> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-carbon/> >>>> >>>> Od: Tomáš Slušný <tomas.slu...@pantheon.tech >>>> <mailto:tomas.slu...@pantheon.tech>> >>>> >>>> Odoslané: 14. októbra 2016 22:13 >>>> >>>> Komu: Luis Gomez; Abhijit Kumbhare >>>> >>>> Kópia: An Ho; openflowplugin-dev; >>>> OpenDayLight-L2switch-Dev >>>> >>>> Predmet: Re: [openflowplugin-dev] [L2switch-dev] Blocker >>>> bug analysis >>>> >>>> >>>> >>>> 6710 was related to both cluster and single node, >>>> because even in single node scenario, we are >>>> registering to ClusterSingletonService, what had problems when >>>> we tried to re-register, when we do not fully >>>> closed previous registration (so, when we disconnected and >>>> reconnected too fast). Most part of this fix was >>>> done in bug 6710, but part of it had to be done also in >>>> openflowplugin side in this gerrit patch: >>>> https://git.opendaylight.org/gerrit/#/c/46390/ >>>> <https://git.opendaylight.org/gerrit/#/c/46390/>. So, Luis, >>>> is it possible to run failing l2switch test on that patch and >>>> see, if it solves anything, or not? >>>> >>>> >>>> >>>> Od: Luis Gomez <ece...@gmail.com >>>> <mailto:ece...@gmail.com>> >>>> >>>> Odoslané: 14. októbra 2016 21:49 >>>> >>>> Komu: Abhijit Kumbhare >>>> >>>> Kópia: An Ho; openflowplugin-dev; >>>> OpenDayLight-L2switch-Dev >>>> >>>> Predmet: Re: [openflowplugin-dev] [L2switch-dev] Blocker >>>> bug analysis >>>> >>>> >>>> >>>> Well I think if openflowplugin people cannot figure out >>>> where the issue is, the next would be to get >>>> some debug and further analysis from l2switch people >>>> explaining the application miss-behavior. >>>> >>>> >>>> >>>> If there are no people available in l2switch to do this, >>>> we can lower the priority to major or critical. >>>> >>>> >>>> >>>> BR/Luis >>>> >>>> >>>> >>>> >>>> >>>>> On Oct 14, 2016, at 12:35 PM, Luis Gomez >>>> <ece...@gmail.com <mailto:ece...@gmail.com>> wrote: >>>> >>>>> >>>> >>>>> As far as I can tell nothing we have done so far has >>>> helped l2switch test, it is still failing. >>>> >>>>> >>>> >>>>> BR/Luis >>>> >>>>> >>>> >>>>>> On Oct 14, 2016, at 9:53 AM, Abhijit Kumbhare >>>> <abhijitk...@gmail.com <mailto:abhijitk...@gmail.com>> >>>> wrote: >>>> >>>>>> >>>> >>>>>> About the first one - >>>> https://bugs.opendaylight.org/show_bug.cgi?id=6575 >>>> <https://bugs.opendaylight.org/show_bug.cgi?id=6575>: >>>> >>>>>> >>>> >>>>>> Tomas has said in yesterday's meeting that this was >>>> dependent on >>>> https://bugs.opendaylight.org/show_bug.cgi?id=6710 >>>> <https://bugs.opendaylight.org/show_bug.cgi?id=6710>. And >>>> that it was fixed. Can we retest? Also a question for Tomas: >>>> 6710 is cluster related - while 6575 is not >>>> clear if it is cluster or single node. Are we sure 6575 is >>>> fixed? >>>> >>>>>> >>>> >>>>>> The second one: >>>> https://bugs.opendaylight.org/show_bug.cgi?id=6917 >>>> <https://bugs.opendaylight.org/show_bug.cgi?id=6917> - Shuva >>>> will respond about that some time. >>>> >>>>>> >>>> >>>>>> >>>> >>>>>> >>>> >>>>>> On Thu, Oct 13, 2016 at 9:50 PM, Luis Gomez >>>> <ece...@gmail.com <mailto:ece...@gmail.com>> wrote: >>>> >>>>>> Hi all, >>>> >>>>>> >>>> >>>>>> We are getting close to Boron SR1 so I think it makes >>>> sense to review the 2 blocking issues we have: >>>> >>>>>> >>>> >>>>>> >>>> >>>>>> 1) https://bugs.opendaylight.org/show_bug.cgi?id=6575 >>>> <https://bugs.opendaylight.org/show_bug.cgi?id=6575> >>>> >>>>>> >>>> >>>>>> Summary: >>>> >>>>>> >>>> >>>>>> l2switch does not work well when mininet is >>>> disconnected and connected with no time in-between. >>>> >>>>>> >>>> >>>>>> Description: >>>> >>>>>> >>>> >>>>>> This is kind of old issue, since the He->Li migration >>>> the l2switch has experienced random issues in >>>> the system test: >>>> >>>>>> >>>> >>>>>> >>>> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/ >>>> >>>> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-boron/> >>>> >>>>>> >>>> >>>>>> Same test passes fine in Beryllium as you can see below: >>>> >>>>>> >>>> >>>>>> >>>> https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-beryllium/ >>>> >>>> <https://jenkins.opendaylight.org/releng/view/l2switch/job/l2switch-csit-1node-switch-only-beryllium/> >>>> >>>>>> >>>> >>>>>> The last discovery (just before Boron release) was that >>>> giving more time between stop mininet and >>>> start mininet made the suite pass. >>>> >>>>>> >>>> >>>>>> Criticality: >>>> >>>>>> >>>> >>>>>> Although this was a clear regression in l2switch test >>>> (Be->B), this bug was not initially marked as >>>> blocker because it was not trivial to reproduce (e.g. switch >>>> connection flap). >>>> >>>>>> >>>> >>>>>> Risk of not fixing: >>>> >>>>>> >>>> >>>>>> l2switch and other similar applications relying on >>>> ofplugin may not work well when switch connection >>>> flaps. >>>> >>>>>> >>>> >>>>>> >>>> >>>>>> 2) https://bugs.opendaylight.org/show_bug.cgi?id=6917 >>>> <https://bugs.opendaylight.org/show_bug.cgi?id=6917> >>>> >>>>>> >>>> >>>>>> Summary: >>>> >>>>>> >>>> >>>>>> Flow matching function (operational flow >>>> reconciliation) is not stable. >>>> >>>>>> >>>> >>>>>> Description: >>>> >>>>>> >>>> >>>>>> I discovered this issue doing some random flow push >>>> test in my laptop using POSTMAN: adding and >>>> deleting the same flow few times produced an alien ID in the >>>> operational flow. >>>> >>>>>> After that I have created a test that does exactly >>>> that: add flow, verify operational ID, delete flow, >>>> sleep 5s, repeat. With these simple steps the issue shows >>>> consistently for Boron (new test): >>>> >>>>>> >>>> >>>>>> >>>> >>>> https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-boron/758/archives/log.html.gz >>>> >>>> <https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-boron/758/archives/log.html.gz> >>>> >>>>>> >>>> >>>>>> But not in Beryllium: >>>> >>>>>> >>>> >>>>>> >>>> >>>> https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-beryllium/1854/archives/log.html.gz >>>> >>>> <https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-beryllium/1854/archives/log.html.gz> >>>> >>>>>> >>>> >>>>>> Criticality: >>>> >>>>>> >>>> >>>>>> Besides the test regression, I think there are >>>> applications in ODL relying on operational flow ID that >>>> would be negatively impacted by this bug. >>>> >>>>>> >>>> >>>>>> Risk of not fixing: >>>> >>>>>> >>>> >>>>>> OF applications relying on operational flow ID (e.g. to >>>> confirm flows) can sporadically fail. >>>> >>>>>> >>>> >>>>>> >>>> >>>>>> _______________________________________________ >>>> >>>>>> L2switch-dev mailing list >>>> >>>>>> L2switch-dev@lists.opendaylight.org >>>> <mailto:L2switch-dev@lists.opendaylight.org> >>>> >>>>>> >>>> https://lists.opendaylight.org/mailman/listinfo/l2switch-dev >>>> <https://lists.opendaylight.org/mailman/listinfo/l2switch-dev> >>>> >>>>>> >>>> >>>>> >>>> >>>> >>>> >>>> TomášSlušný >>>> >>>> Software Developer >>>> >>>> >>>> >>>> Sídlo / Mlynské Nivy 56 / 821 05 Bratislava / Slovakia >>>> >>>> R&D centrum / Janka Kráľa 9 / 974 01 Banská Bystrica / >>>> Slovakia >>>> >>>> +421 911 083 902 <tel:%2B421%20911%20083%20902> / >>>> tomas.slu...@pantheon.tech >>>> <mailto:tomas.slu...@pantheon.tech> >>>> >>>> reception: +421 2 206 65 114 >>>> <tel:%2B421%202%20206%2065%20114> / www.pantheon.sk >>>> <http://www.pantheon.sk/> >>>> >>>> >>>> >>>> >>>> >>>> TomášSlušný >>>> >>>> Software Developer >>>> >>>> >>>> >>>> Sídlo / Mlynské Nivy 56 / 821 05 Bratislava / Slovakia >>>> >>>> R&D centrum / Janka Kráľa 9 / 974 01 Banská Bystrica / >>>> Slovakia >>>> >>>> +421 911 083 902 <tel:%2B421%20911%20083%20902> / >>>> tomas.slu...@pantheon.tech >>>> <mailto:tomas.slu...@pantheon.tech> >>>> >>>> reception: +421 2 206 65 114 >>>> <tel:%2B421%202%20206%2065%20114> / www.pantheon.sk >>>> <http://www.pantheon.sk/> >>>> >>>> >>>> >>>> >>>> >>> >>>> >> >>>> >> _______________________________________________ >>>> >> openflowplugin-dev mailing list >>>> >> openflowplugin-...@lists.opendaylight.org >>>> <mailto:openflowplugin-...@lists.opendaylight.org> >>>> >> >>>> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev >>>> >>>> <https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev> >>>> >> MiroslavMacko >>>> >> Software Developer >>>> >> >>>> >> Sídlo / Mlynské Nivy 56 / 821 05 Bratislava / Slovakia >>>> >> R&D centrum / Janka Kráľa 9 / 974 01 Banská Bystrica / >>>> Slovakia >>>> >> / miroslav.ma...@pantheon.tech >>>> <mailto:miroslav.ma...@pantheon.tech> >>>> >> reception: +421 2 206 65 114 >>>> <tel:%2B421%202%20206%2065%20114> / www.pantheon.sk >>>> <http://www.pantheon.sk/> >>>> >> >>>> >> [logo] >>>> >> >>>> >> >>>> > >>>> >>>> _______________________________________________ >>>> L2switch-dev mailing list >>>> L2switch-dev@lists.opendaylight.org >>>> <mailto:L2switch-dev@lists.opendaylight.org> >>>> https://lists.opendaylight.org/mailman/listinfo/l2switch-dev >>>> <https://lists.opendaylight.org/mailman/listinfo/l2switch-dev> >>>> >>>> >>> >>> >> > > > > > _______________________________________________ > L2switch-dev mailing list > L2switch-dev@lists.opendaylight.org > https://lists.opendaylight.org/mailman/listinfo/l2switch-dev > _______________________________________________ L2switch-dev mailing list L2switch-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/l2switch-dev