Addressed them ☺

Thanks
Shuva

From: Anil Vishnoi [mailto:[email protected]]
Sent: Thursday, June 16, 2016 5:26 AM
To: Shuva Jyoti Kar
Cc: Luis Gomez; [email protected]
Subject: Re: [openflowplugin-dev] BUG 6064

Shuva, please address the comments from Bertrand on your patch. Otherwise patch 
looks good.

Thanks
Anil

On Tue, Jun 14, 2016 at 9:28 PM, Shuva Jyoti Kar 
<[email protected]<mailto:[email protected]>> wrote:
Thanks a ton Luis. Sorry Miro and all for the unexpected failures.

@ Luis can you do a +1 on the patch, so that I can get Abhijit/Anil to review 
and merge it.

Thanks
Shuva

From: Luis Gomez [mailto:[email protected]<mailto:[email protected]>]
Sent: Wednesday, June 15, 2016 9:52 AM
To: Shuva Jyoti Kar
Cc: Miroslav Macko; 
[email protected]<mailto:[email protected]>
Subject: Re: BUG 6064

OK, I confirmed both: the regression was introduced by DCTN patch and this last 
patch fixes the regression.

BR/Luis


On Jun 14, 2016, at 6:41 PM, Shuva Jyoti Kar 
<[email protected]<mailto:[email protected]>> wrote:

Hi Miro,

Can you please test it out with this patch 
https://git.opendaylight.org/gerrit/#/c/40346 and let us know

Thanks
Shuva

From: Luis Gomez [mailto:[email protected]]
Sent: Wednesday, June 15, 2016 6:07 AM
To: Miroslav Macko
Cc: 
[email protected]<mailto:[email protected]>;
 Shuva Jyoti Kar
Subject: Re: BUG 6064

Thanks Miro for pointing this, by looking at the logs I would say it is the 
DTCN change, I am running some test in CI to confirm, unfortunately the Jenkins 
queue is kind of busy now so it will take a while to verify.

It is also unfortunate daily jobs like 100K flows test are not normally used to 
test individual patches. Whenever my time allows I will create a keyword that 
will launch project related csit tests on a given patch.

BR/Luis



On Jun 14, 2016, at 12:11 PM, Miroslav Macko 
<[email protected]<mailto:[email protected]>> wrote:

Hi guys,

I have created bug https://bugs.opendaylight.org/show_bug.cgi?id=6064

@Shuva could you please have a quick look, if there can be problem?

@Luis is it possible somehow to check which change caused the problem?

Thanks a lot,
Miro
MiroslavMacko
Software Developer

Sídlo / Mlynské Nivy 56 / 821 05 Bratislava / Slovakia
R&D centrum / Janka Kráľa 9 /  974 01 Banská Bystrica / Slovakia
/ [email protected]<mailto:[email protected]>
reception: +421 2 206 65 114 / www.pantheon.sk<http://www.pantheon.sk/>
[logo]


_______________________________________________
openflowplugin-dev mailing list
[email protected]<mailto:[email protected]>
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev



--
Thanks
Anil
_______________________________________________
openflowplugin-dev mailing list
[email protected]
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Reply via email to