Hi Everyone, (I've changed the subject as we have a better idea of the issue now)
We can't have projects broken for so long. Autorelease Boron has been broken since April 27th which is almost 1 month now. What's the status here? who is actively working on this? Yes, changes to OFP broke this. Do OFP committers need to assist here to get this resolved? I believe NIC boron is also broke due to this same issue so adding them to this thread. Regards, Thanh On 11 May 2016 at 04:32, Jozef Bacigal -X (jbacigal - PANTHEON TECHNOLOGIES at Cisco) <jbaci...@cisco.com> wrote: > Hi all > > > > Can I ask you gyus what exactly are the expected result in operational DS? > You think it was caused by sny changes in ofp ? > > > > Jozef > > > > *From:* openflowplugin-dev-boun...@lists.opendaylight.org [mailto: > openflowplugin-dev-boun...@lists.opendaylight.org] *On Behalf Of *Manikantan, > Anandhi > *Sent:* 11. mája 2016 2:38 > *To:* Abhijit Kumbhare <abhijitk...@gmail.com> > *Cc:* didm-...@lists.opendaylight.org; Thanh Ha < > thanh...@linuxfoundation.org>; rele...@lists.opendaylight.org; > openflowplugin-dev <openflowplugin-dev@lists.opendaylight.org>; Subhash > Singh <subhash_si...@criterionnetworks.com> > > *Subject:* 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, the adjust flow feature > is broken now in DIDM. > > Attached please find the output for comparison. > > Also, the instructions are missing in table 200. > > > > Since I’m busy with other projects right now, won’t be able to spend much > time. But would be great if the oF plugin team can revise the changes and > take action accordingly. > > > > Thanks > > Anandhi > > *From:* Abhijit Kumbhare [mailto:abhijitk...@gmail.com > <abhijitk...@gmail.com>] > *Sent:* Monday, May 09, 2016 1:54 PM > *To:* Manikantan, Anandhi <anandhi.manikan...@hpe.com> > *Cc:* Subhash Singh <subhash_si...@criterionnetworks.com>; Thanh Ha < > thanh...@linuxfoundation.org>; openflowplugin-dev < > openflowplugin-dev@lists.opendaylight.org>; rele...@lists.opendaylight.org; > didm-...@lists.opendaylight.org > *Subject:* Re: [openflowplugin-dev] [release] [didm-dev] didm-hp module > failing to compile > > > > 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 > >
_______________________________________________ openflowplugin-dev mailing list openflowplugin-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev