Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-11 Thread Jozef Bacigal -X (jbacigal - PANTHEON TECHNOLOGIES at Cisco)
. mája 2016 2:38 To: Abhijit Kumbhare Cc: didm-...@lists.opendaylight.org; Thanh Ha ; rele...@lists.opendaylight.org; openflowplugin-dev ; Subhash Singh Subject: Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile Since Subhash didn’t have HP switches, I’ve

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-10 Thread Manikantan, Anandhi
ubject: RE: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile Since Subhash didn’t have HP switches, I’ve recreated the setup and tested today. I’m not getting the expected output with the changes made in the code review. Since the methods are modified in openflow plugin,

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-10 Thread Manikantan, Anandhi
...@gmail.com] Sent: Monday, May 09, 2016 1:54 PM To: Manikantan, Anandhi Cc: Subhash Singh ; Thanh Ha ; openflowplugin-dev ; rele...@lists.opendaylight.org; didm-...@lists.opendaylight.org Subject: Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile Thanks Alexis & Ana

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-09 Thread Abhijit Kumbhare
Thanks Alexis & Anandhi! On Mon, May 9, 2016 at 1:37 PM, Manikantan, Anandhi < anandhi.manikan...@hpe.com> wrote: > Hi Subhash, > > > > I reviewed the code and comments. Alex has responded to the comments and > feels that this needs to be tested before merge. > > Since I don’t have the setup now,

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-09 Thread Manikantan, Anandhi
Hi Subhash, I reviewed the code and comments. Alex has responded to the comments and feels that this needs to be tested before merge. Since I don’t have the setup now, would you be able to test and let me know so that I can merge. I can help you with test guidelines if needed Please shoot me an

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-06 Thread Jamo Luhrsen
Thanks for the help, Alexis. On 05/06/2016 05:45 AM, Alexis de Talhouët wrote: > And the candidate fix: > https://git.opendaylight.org/gerrit/#/c/38507/ > > Thanks, > Alexis > >> On May 6, 2016, at 8:21 AM, Alexis de Talhouët > > wrote: >> >> FYI, this is the reas

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-06 Thread Subhash Singh
Hello Anandhi, Could you please review the patch, as you are more familiar with this part of the implementation and also I don't have the rights to merge. -- Thanks and Regards, Subhash Kumar Singh On Fri, May 6, 2016 at 7:27 PM, Thanh Ha wrote: > Thanks Alexis, > > Can someone review and merg

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-06 Thread Thanh Ha
Thanks Alexis, Can someone review and merge the patch? Thanks, Thanh On 6 May 2016 at 08:45, Alexis de Talhouët wrote: > And the candidate fix: > https://git.opendaylight.org/gerrit/#/c/38507/ > > Thanks, > Alexis > > On May 6, 2016, at 8:21 AM, Alexis de Talhouët > wrote: > > FYI, this is th

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-06 Thread Alexis de Talhouët
And the candidate fix: https://git.opendaylight.org/gerrit/#/c/38507/ Thanks, Alexis > On May 6, 2016, at 8:21 AM, Alexis de Talhouët > wrote: > > FYI, this is the reason one DIDM is broken: changed of OFP yang model: > > https://git.opendayli

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-06 Thread Alexis de Talhouët
FYI, this is the reason one DIDM is broken: changed of OFP yang model: https://git.opendaylight.org/gerrit/#/c/36559/5/model/model-flow-service/src/main/yang/flow-node-inventory.yang

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-05 Thread Thanh Ha
FYI we've disabled DIDM from autorelease-boron so that the job can continue testing until this bug is sorted out. Regards, Thanh On 5 May 2016 at 16:39, Thanh Ha wrote: > To be clear it's the root cause of why we can't reproduce the failure > locally since ofp hasn't been pushing to Nexus their

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-05 Thread Abhijit Kumbhare
Merged Luis' changes (except in stable/beryllium where it is just ready to submit due to the branch lock). On Thu, May 5, 2016 at 12:18 PM, Luis Gomez wrote: > https://git.opendaylight.org/gerrit/#/c/37757/ > > https://git.opendaylight.org/gerrit/#/c/37758/ > https://git.opendaylight.org/gerrit/

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-05 Thread Thanh Ha
To be clear it's the root cause of why we can't reproduce the failure locally since ofp hasn't been pushing to Nexus their latest artifacts. didm-hp will still fail to compile so we still need someone to look into why getTableFeature() is failing. Regards, Thanh On 5 May 2016 at 15:11, Jamo Luhr

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-05 Thread Luis Gomez
https://git.opendaylight.org/gerrit/#/c/37757 / https://git.opendaylight.org/gerrit/#/c/37758/ https://git.opendaylight.org/gerrit/#/c/37759/ > On May 5, 2016, at 12:16 PM, Luis Gomez wrote: > > I patched 3 branches of ofplugin some time ago but no

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-05 Thread Luis Gomez
I patched 3 branches of ofplugin some time ago but nobody merged it: https://git.opendaylight.org/gerrit/#/c/37759/ https://git.opendaylight.org/gerrit/#/c/37758/ https://git.opendaylight.org/gerrit

Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module failing to compile

2016-05-05 Thread Jamo Luhrsen
after more debugging with Thanh, we think the root cause is because openflowplugin's merge job started failing because of the http vs https issue when uploading artifacts to nexus. this patch in openflowplugin should fix that: https://git.opendaylight.org/gerrit/38436 JamO On 05/05/2016 11:38