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, 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 e-mail offline. > > > > Thanks > > Anandhi > > > > *From:* Subhash Singh [mailto:subhash_si...@criterionnetworks.com] > *Sent:* Friday, May 06, 2016 8:08 AM > *To:* Manikantan, Anandhi <anandhi.manikan...@hpe.com> > *Cc:* Alexis de Talhouët <adetalho...@inocybe.com>; Jamo Luhrsen < > jluhr...@gmail.com>; rele...@lists.opendaylight.org; > didm-...@lists.opendaylight.org; openflowplugin-dev < > openflowplugin-dev@lists.opendaylight.org>; Thanh Ha < > thanh...@linuxfoundation.org> > *Subject:* Re: [release] [didm-dev] didm-hp module failing to compile > > > > 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 <thanh...@linuxfoundation.org> > wrote: > > Thanks Alexis, > > > > Can someone review and merge the patch? > > > > Thanks, > > Thanh > > > > > > On 6 May 2016 at 08:45, Alexis de Talhouët <adetalho...@inocybe.com> > 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 <adetalho...@inocybe.com> > wrote: > > > > 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 > > > > Thanks, > > Alexis > > > > On May 5, 2016, at 8:03 PM, Thanh Ha <thanh...@linuxfoundation.org> wrote: > > > > 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 <thanh...@linuxfoundation.org> 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 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 Luhrsen <jluhr...@gmail.com> wrote: > > 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 AM, Subhash Singh wrote: > > +[Anandhi] > > > > Hello Thanh, > > > > Anandhi is familiar with didm-hp module (using getTableFeature), I think > she could add clarification on this issue. > > > > -- > > Regards, > > Subhash Kumar Singh > > > > On Thu, May 5, 2016 at 11:46 PM, Thanh Ha <thanh...@linuxfoundation.org > <mailto:thanh...@linuxfoundation.org>> wrote: > > > > On 5 May 2016 at 00:31, Thanh Ha <thanh...@linuxfoundation.org > <mailto:thanh...@linuxfoundation.org>> wrote: > > > > Hi didm-devs, > > > > Autorelease has been failing since April 27th due to didm-hp > module failing to compile due to some unresolvable symbols (missing or > changed > > dependencies?). Can someone take a look? > > > > I opened bug 5843 to track this issue [0]. > > > > Regards, > > Thanh > > > > [0] https://bugs.opendaylight.org/show_bug.cgi?id=5843 > > > > > > > > Hi Everyone, > > > > So taking a look at this closer with Jamo just now. We noticed that > this patch [0] in openflowplugin removed something that sounds like > > getTableFeatures() but I'm not at all familiar with how yang maps to > Java so I could be wrong but could this be the cause of getTableFeatures() > > being missing? > > > > If so how do we fix this? > > > > Regards, > > Thanh > > > > [0] > https://git.opendaylight.org/gerrit/#/c/36559/5/model/model-flow-statistics/src/main/yang/opendaylight-flow-table-statistics.yang > > > > _______________________________________________ > > didm-dev mailing list > > didm-...@lists.opendaylight.org <mailto: > didm-...@lists.opendaylight.org> > > https://lists.opendaylight.org/mailman/listinfo/didm-dev > > > > > > > > > > _______________________________________________ > > release mailing list > > rele...@lists.opendaylight.org > > https://lists.opendaylight.org/mailman/listinfo/release > > > > > > > > _______________________________________________ > release mailing list > rele...@lists.opendaylight.org > https://lists.opendaylight.org/mailman/listinfo/release > > > > > > > > > > _______________________________________________ > openflowplugin-dev mailing list > openflowplugin-dev@lists.opendaylight.org > https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev > >
_______________________________________________ openflowplugin-dev mailing list openflowplugin-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
