Anyway Luis, Maybe I can’t read the proper the tests, but how do you see there is regression ? :)
Jozef From: Jozef Bacigál [mailto:[email protected]] Sent: Tuesday, March 28, 2017 8:30 AM To: Abhijit Kumbhare <[email protected]>; Luis Gomez <[email protected]>; openflowplugin-dev <[email protected]>; Tomáš Slušný <[email protected]> Subject: Re: [openflowplugin-dev] Last patch in boron introduces perf regression Abhijit so in the master branch no problem we got plenty of time, but you say for the SR3 better with race condition than regression ? P.S.: Right now all merges are blocked assume so we can only prepare revert. Here is the revert for boron: https://git.opendaylight.org/gerrit/#/c/53953/ Jozef. From: Abhijit Kumbhare [mailto:[email protected]] Sent: Tuesday, March 28, 2017 7:21 AM To: Luis Gomez <[email protected]<mailto:[email protected]>>; openflowplugin-dev <[email protected]<mailto:[email protected]>> Subject: Re: [openflowplugin-dev] Last patch in boron introduces perf regression Sounds good. Jozef/Tomas can you do it? I assume the race condition that the patch fixed was pre-existing. On Mon, Mar 27, 2017 at 9:12 PM Luis Gomez <[email protected]<mailto:[email protected]>> wrote: Hi all, After looking these boron jobs: https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-periodic-scale-stats-collection-daily-only-boron/ https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-3node-periodic-bulkomatic-clustering-daily-only-boron/ and testing SR3 manually, I conclude last patch we merged to fix stats issues: https://git.opendaylight.org/gerrit/#/c/53545 Introduces *important* perf regression, so I really recommend to revert it before we release SR3. Same thing I recommend for master but there we have more time to revert/fix: https://git.opendaylight.org/gerrit/#/c/52237 BR/Luis _______________________________________________ openflowplugin-dev mailing list [email protected]<mailto:[email protected]> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
