It is not obvious but this test for example started to fail after the merge:

https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-3node-periodic-bulkomatic-clustering-daily-only-boron/310/robot/openflowplugin-bulkomatic-clustering-daily.txt/Cluster%20Reconcilliation%20Multi%20DPN/Verify%20Flows%20In%20Switch%20Before%20Cluster%20Restart/

Which means we cannot push 10K flows in a single switch. I did more test 
locally like pushing 20K in 8 switches and that also does not work in current 
SR3.

BR/Luis


> On Mar 28, 2017, at 12:33 AM, Jozef Bacigál <[email protected]> 
> wrote:
> 
> 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/ 
> <https://git.opendaylight.org/gerrit/#/c/53953/>
>  
> Jozef.
>  
>  
> From: Abhijit Kumbhare [mailto:[email protected] 
> <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-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/
>  
> <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 
> <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 
> <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 
> <https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev>
_______________________________________________
openflowplugin-dev mailing list
[email protected]
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Reply via email to