Thanks! On Tuesday, May 24, 2016, Anil Vishnoi <vishnoia...@gmail.com> wrote:
> So looks like these yang model changes happened and only lithium plugin > was adapted to these changes. Helium plugin is still not adapted to these > changes and that's why NIC/DIDM projects are probably not able to find the > table features and failing. I am looking into it, and push the patch as > soon as i fix it. > > On Tue, May 24, 2016 at 4:29 PM, Abhijit Kumbhare <abhijitk...@gmail.com > <javascript:_e(%7B%7D,'cvml','abhijitk...@gmail.com');>> wrote: > >> Did not see this thread before. Added Jozef & Michal. I believe Alexis >> has narrowed the breakage to: >> >> >> >> https://git.opendaylight.org/gerrit/#/c/36559/5/model/model-flow-service/src/main/yang/flow-node-inventory.yang >> >> >> >> On Tue, May 24, 2016 at 1:53 PM, Manikantan, Anandhi < >> anandhi.manikan...@hpe.com >> <javascript:_e(%7B%7D,'cvml','anandhi.manikan...@hpe.com');>> wrote: >> >>> Please see my response attached in this thread. >>> >>> >>> >>> -Anandhi >>> >>> >>> >>> *From:* Subhash Singh [mailto:subhash_si...@criterionnetworks.com >>> <javascript:_e(%7B%7D,'cvml','subhash_si...@criterionnetworks.com');>] >>> *Sent:* Tuesday, May 24, 2016 1:48 PM >>> *To:* Thanh Ha <thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','thanh...@linuxfoundation.org');>> >>> *Cc:* didm-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','didm-...@lists.opendaylight.org');>; >>> openflowplugin-dev <openflowplugin-dev@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','openflowplugin-dev@lists.opendaylight.org');>>; >>> nic-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','nic-...@lists.opendaylight.org');>; >>> rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');>; >>> Robert Varga <n...@hq.sk <javascript:_e(%7B%7D,'cvml','n...@hq.sk');>>; >>> Manikantan, Anandhi <anandhi.manikan...@hpe.com >>> <javascript:_e(%7B%7D,'cvml','anandhi.manikan...@hpe.com');>> >>> *Subject:* Re: [release] [openflowplugin-dev] [didm-dev] [nic-dev] OFP >>> getTableFeature() breakage in nic and didm >>> >>> >>> >>> Hello Thanh Ha, >>> >>> >>> >>> Yes I think OFP committers need to assist us to fix this issue. >>> >>> Currently, the submitted patch [1] is not producing the response as >>> expected by DIDM. >>> >>> >>> >>> @Anandhi, >>> >>> Could you please add more clarity on it. >>> >>> >>> >>> [1] https://git.opendaylight.org/gerrit/#/c/38507/ >>> >>> >>> >>> -- >>> >>> Regards, >>> >>> Subhash Kumar Singh >>> >>> >>> >>> >>> >>> >>> >>> On Wed, May 25, 2016 at 12:43 AM, Robert Varga <n...@hq.sk >>> <javascript:_e(%7B%7D,'cvml','n...@hq.sk');>> wrote: >>> >>> On 05/24/2016 04:50 PM, Thanh Ha wrote: >>> > I believe NIC boron is also broke due to this same issue so adding them >>> > to this thread. >>> >>> Yes, it has not published artifacts for a long time, those got purged >>> and now we cannot publish new ones via remerge. >>> >>> Bye, >>> Robert >>> >>> >>> >>> >>> ---------- Forwarded message ---------- >>> From: "Manikantan, Anandhi" <anandhi.manikan...@hpe.com >>> <javascript:_e(%7B%7D,'cvml','anandhi.manikan...@hpe.com');>> >>> To: Abhijit Kumbhare <abhijitk...@gmail.com >>> <javascript:_e(%7B%7D,'cvml','abhijitk...@gmail.com');>> >>> Cc: Subhash Singh <subhash_si...@criterionnetworks.com >>> <javascript:_e(%7B%7D,'cvml','subhash_si...@criterionnetworks.com');>>, >>> Thanh Ha <thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','thanh...@linuxfoundation.org');>>, >>> openflowplugin-dev <openflowplugin-dev@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','openflowplugin-dev@lists.opendaylight.org');>>, >>> "rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');>" < >>> rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');>>, " >>> didm-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','didm-...@lists.opendaylight.org');>" < >>> didm-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','didm-...@lists.opendaylight.org');>> >>> Date: Wed, 11 May 2016 00:38:05 +0000 >>> 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 >>> <javascript:_e(%7B%7D,'cvml','abhijitk...@gmail.com');>] >>> *Sent:* Monday, May 09, 2016 1:54 PM >>> *To:* Manikantan, Anandhi <anandhi.manikan...@hpe.com >>> <javascript:_e(%7B%7D,'cvml','anandhi.manikan...@hpe.com');>> >>> *Cc:* Subhash Singh <subhash_si...@criterionnetworks.com >>> <javascript:_e(%7B%7D,'cvml','subhash_si...@criterionnetworks.com');>>; >>> Thanh Ha <thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','thanh...@linuxfoundation.org');>>; >>> openflowplugin-dev <openflowplugin-dev@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','openflowplugin-dev@lists.opendaylight.org');>>; >>> rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');>; >>> didm-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','subhash_si...@criterionnetworks.com');>] >>> *Sent:* Friday, May 06, 2016 8:08 AM >>> *To:* Manikantan, Anandhi <anandhi.manikan...@hpe.com >>> <javascript:_e(%7B%7D,'cvml','anandhi.manikan...@hpe.com');>> >>> *Cc:* Alexis de Talhouët <adetalho...@inocybe.com >>> <javascript:_e(%7B%7D,'cvml','adetalho...@inocybe.com');>>; Jamo >>> Luhrsen <jluhr...@gmail.com >>> <javascript:_e(%7B%7D,'cvml','jluhr...@gmail.com');>>; >>> rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');>; >>> didm-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','didm-...@lists.opendaylight.org');>; >>> openflowplugin-dev <openflowplugin-dev@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','openflowplugin-dev@lists.opendaylight.org');>>; >>> Thanh Ha <thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','thanh...@linuxfoundation.org');> <mailto: >>> thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','thanh...@linuxfoundation.org');>>> wrote: >>> > >>> > On 5 May 2016 at 00:31, Thanh Ha <thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','thanh...@linuxfoundation.org');> <mailto: >>> thanh...@linuxfoundation.org >>> <javascript:_e(%7B%7D,'cvml','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 >>> <javascript:_e(%7B%7D,'cvml','didm-...@lists.opendaylight.org');> >>> <mailto:didm-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','didm-...@lists.opendaylight.org');>> >>> > https://lists.opendaylight.org/mailman/listinfo/didm-dev >>> > >>> > >>> > >>> > >>> > _______________________________________________ >>> > release mailing list >>> > rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');> >>> > https://lists.opendaylight.org/mailman/listinfo/release >>> > >>> >>> >>> >>> >>> >>> _______________________________________________ >>> release mailing list >>> rele...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','rele...@lists.opendaylight.org');> >>> https://lists.opendaylight.org/mailman/listinfo/release >>> >>> >>> >>> >>> >>> >>> >>> >>> >>> >>> _______________________________________________ >>> openflowplugin-dev mailing list >>> openflowplugin-dev@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','openflowplugin-dev@lists.opendaylight.org');> >>> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev >>> >>> >>> >>> _______________________________________________ >>> nic-dev mailing list >>> nic-...@lists.opendaylight.org >>> <javascript:_e(%7B%7D,'cvml','nic-...@lists.opendaylight.org');> >>> https://lists.opendaylight.org/mailman/listinfo/nic-dev >>> >>> >> >> _______________________________________________ >> openflowplugin-dev mailing list >> openflowplugin-dev@lists.opendaylight.org >> <javascript:_e(%7B%7D,'cvml','openflowplugin-dev@lists.opendaylight.org');> >> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev >> >> > > > -- > Thanks > Anil >
_______________________________________________ openflowplugin-dev mailing list openflowplugin-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
