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

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,

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:

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