Re: [sfc-dev] [release] [WEATHER] Genius ActionInfo/InstructionInfo/MatchInfo clean-up

2017-01-24 Thread Robert Varga
On 01/24/2017 04:28 PM, Stephen Kitt wrote: > Hi, > > All the affected projects are already aware of this, but as a general > heads-up, the Genius project will be dropping the old ActionInfo, > InstructionInfo and MatchInfo classes in Carbon (in particular, > ActionType, InstructionType and

Re: [sfc-dev] [genius-dev] [release] [WEATHER] Genius ActionInfo/InstructionInfo/MatchInfo clean-up

2017-01-24 Thread Robert Varga
On 01/24/2017 06:17 PM, Michael Vorburger wrote: > One question around this: these look like hand-written APIs, which > really should be backed by a model. Is there a particular reason for > this? > > > using https://immutables.github.io for these could be interesting.. but >

Re: [sfc-dev] Build failure on carbon master due to yang-to-sources error

2017-02-28 Thread Robert Varga
On 02/28/2017 10:20 AM, Brady Allen Johnson wrote: > > Robert, > > I didnt include the full stack trace since it was in the Jenkins link > originally provided, and I didnt want to clutter the email, but here it is: Ah, sorry to have missed it. > > *06:19:02* [INFO] Adding ModuleInfo providers

Re: [sfc-dev] [mdsal-dev] Real problem or misleading exception with mandatory

2016-09-07 Thread Robert Varga
On 09/07/2016 01:00 PM, Yang, Yi Y wrote: > I also ever encountered this issue, but current master (without this fix) can > work without any problem, I'm wondering why "presence" can fix this, isn't > this yang-tools issue? Any augmentation should be optional but not mandatory. > > Does every

Re: [sfc-dev] [controller-dev] ODL Boron for Load Balancer Scenaio

2017-07-17 Thread Robert Varga
On 17/07/17 16:03, Rahul Hada wrote: > Hi Robert, > > So, can you suggest a way to LBaaS scenario for OpenDayLight Boron SR2? Hello, sorry, openflow-based apps are not my area of expertise, so I cannot offer further help on this topic. Regards, Robert signature.asc Description: OpenPGP

Re: [sfc-dev] [controller-dev] ODL Boron for Load Balancer Scenaio

2017-07-17 Thread Robert Varga
On 17/07/17 12:52, Jaime Caamaño Ruiz wrote: > Hello Rahul > > The load balancer you reference seems to be an old ODL controller > sample application that might no longer be available. Copying > controller-dev in case anyone there can contribute. That looks like an old AD-SAL-based app. Since

Re: [sfc-dev] [release] autorelease oxygen broken due to SFC

2018-06-06 Thread Robert Varga
On 06/06/18 13:33, Michael Vorburger wrote: > Hello project SFC, > > according > to  > https://jenkins.opendaylight.org/releng/job/controller-validate-autorelease-oxygen/346/console > the autorelease oxygen is broken due to something in your project, check > this out - will you reply here when

Re: [sfc-dev] [release] Autorelease oxygen failed to build odl-sfc-scf-openflow from sfc

2018-05-31 Thread Robert Varga
On 31/05/18 22:05, Jamo Luhrsen wrote: > this is exactly one of the reasons it seems to make sense to just disable > SFT from autorelease. I have no hard data, but I would guess that Brady's > response below is what we get in the overwhelming majority of these > failures. > > so we re-run,

Re: [sfc-dev] [release] [OpenDaylight TSC] ietf-interfaces and ietf-ip upgrades

2018-07-02 Thread Robert Varga
On 01/07/18 07:36, Ariel Adam wrote: > Robert, does this effect any existing DBs or external APIs? > Can you think of any possible back compatibility issues that should be > addressed? Compatibility-wise, there are four aspects: 1) updated revision, meaning a datastore fixup is require during

Re: [sfc-dev] [OpenDaylight TSC] ietf-interfaces and ietf-ip upgrades

2018-07-02 Thread Robert Varga
On 29/06/18 19:55, Lori Jakab wrote: > but there may be other effects, especially in terms of interoperability > with VPP, so this needs careful consideration. I agree. For that purpose we'll be shipping updated NMDA models from MDSAL, but not package them by default. fd.io can then pick them up

Re: [sfc-dev] [release] Autorelease oxygen failed to build sfc-ui-module from sfc

2018-08-14 Thread Robert Varga
On 14/08/18 07:20, Ariel Adam wrote: > Guys, still waiting for more info on why the autorelease failed 2 days ago. [INFO] bower ng-table#* download https://github.com/esvit/ng-table/archive/v0.3.3.zip [...] [ERROR] bower ng-table#* EHTTP

Re: [sfc-dev] [mdsal-dev] [openflowplugin-dev] IncorrectNestingException on SFC when sing OFP model

2018-07-08 Thread Robert Varga
On 07/07/18 06:20, Vishal Thapar wrote: > It could be specific to the action - > NxActionRegLoadNodesNodeTableFlowApplyActionsCase If no one outside of > SFC is using this action, possible the yang definition for this one was > incorrect and some recent patch is now catching the violation. I

Re: [sfc-dev] [mdsal-dev] [openflowplugin-dev] IncorrectNestingException on SFC when sing OFP model

2018-07-08 Thread Robert Varga
On 09/07/18 01:26, Robert Varga wrote: > On 07/07/18 06:20, Vishal Thapar wrote: >> possible the yang definition for this one was incorrect and some recent >> patch is now catching the violation. > output { > uses action:action-list; > } >

Re: [sfc-dev] [openflowplugin-dev] [mdsal-dev] IncorrectNestingException on SFC when sing OFP model

2018-07-10 Thread Robert Varga
On 10/07/18 11:56, Jaime Caamaño Ruiz wrote: > Hello Tom, Robert > >> However the serialization bypass was "broken" in the >> process. I think this explains the difference. > Are patches [1] and [2] intention to restore this bypass? > I tested [3] a multipatch build [4] with both patches and

Re: [sfc-dev] [openflowplugin-dev] [mdsal-dev] IncorrectNestingException on SFC when sing OFP model

2018-07-09 Thread Robert Varga
On 09/07/18 11:55, Faseela K wrote: > Netvirt uses blueprint wiring and injects the odlInterfaceRpcService, where > as sfc uses interfaceManagerRpcService = > rpcProviderRegistry.getRpcService(OdlInterfaceRpcService.class); > > Robert indicated that so netvirt is bypassing MD-SAL, as it is

Re: [sfc-dev] [release] Autorelease nitrogen failed to build sfc-openflow-utils from sfc

2018-02-26 Thread Robert Varga
On 26/02/18 16:35, Brady Johnson wrote: > I cant reproduce this locally with the latest Nitrogen artifacts today. > I'd like to wait until tonight's build to see if this is a transient > failure. Hello Brady, It is not, you are asserting exception on failure message strings. These have changed

Re: [sfc-dev] [release] Autorelease nitrogen failed to build sfc-openflow-utils from sfc

2018-02-27 Thread Robert Varga
On 27/02/18 14:37, Jaime Caamaño Ruiz wrote: > Robert, this will also affect oxygen/fluorine right? Hello Jaime, yes, they will, but that will happen post-oxygen release, once yangtools-2.0.2 comes down the pipeline. Regards, Robert signature.asc Description: OpenPGP digital signature

Re: [sfc-dev] [release] Autorelease nitrogen failed to build sfc-openflow-utils from sfc

2018-02-26 Thread Robert Varga
On 26/02/18 17:02, Robert Varga wrote: > Going forward generated code will need to learn the difference between > the enforcement pattern (which has changed) and the declared pattern, > which does not include the wrapping and should be quoted in the error > string. That will pr

Re: [sfc-dev] [release] TSC Vote for moving SFC to a self managed project

2019-09-17 Thread Robert Varga
On 16/09/2019 06:34, Abhijit Kumbhare wrote: > Hello TSC, > > There was a discussion in the TSC meeting to move SFC project to be a > self managed project for Sodium as SFC members have mostly moved on and > have also not attended any TSC meetings. Subsequently we also sent an > email to the SFC

Re: [sfc-dev] [integration-dev] SFC and Sodium

2019-09-17 Thread Robert Varga
On 13/09/2019 00:39, Abhijit Kumbhare wrote: > Hi SFC team, [+ direct emails to Jaime and David] > During the TSC meeting today we discussed this issue as well as SFC in > general. We decided to remove the SFC dependency and SFC tests from the > NetVirt project. However, I would like to ask a