Re: [vpp-dev] Facing issue with cli-listen option in vpp.conf

2022-07-15 Thread Amaresh Parida
Hi All, Any inputs on the below query will be very helpful. Thanks, Amaresh On Fri, Jul 15, 2022, 14:42 Amaresh Parida via lists.fd.io wrote: > Hi All > > This is regarding the cli-listen option in vpp.conf. > I want to use* cli-listen localhost:5002* instead of *cli-listen > /run/vpp/cli.sock

Re: [vpp-dev] vpp api versioning

2022-07-15 Thread Matthew Smith via lists.fd.io
Hi Andrew, Neale and I are the maintainers of linux-cp. I am ok with changing it in place because the use of "namespace" is preventing Stanislav from even being able to compile his code. When you say "mark the APIs as experimental" are you talking about putting "state: experimental" in the FEATUR

Re: [vpp-dev] vpp api versioning

2022-07-15 Thread Stanislav Zaikin
Hi Pim, There's also lcp_default_ns_get broken. I fixed that locally and want to send the patch. Can take a look at lcp_itf_pair_add_del_v2 as well. On Fri, 15 Jul 2022 at 12:57, Pim van Pelt wrote: > Hoi, > > Currently, linux-cp API doesn't seem to work, at least in Python and GoVPP > (see pr

Re: [vpp-dev] vpp api versioning

2022-07-15 Thread Stanislav Zaikin
Hi Andrew, Thanks for the explanation. I'll try to reach maintainers of the lcp plugin and convince them :) Yes, I'm working on an agent for managing vpp through binary api and it's written in C++ - so it breaks the compilation. I have a patch in my local repo for that, but now I'm trying to merge

Re: [vpp-dev] vpp api versioning

2022-07-15 Thread Pim van Pelt
Hoi, Currently, linux-cp API doesn't seem to work, at least in Python and GoVPP (see previous threads on this mailinglist with bugreports), lcp_itf_pair_add_del_v2() eventually succeeds but throws an error to the caller. This snippet: r = vpp.api.create_loopback() print(r) host_if_name = "loop

Re: [vpp-dev] vpp api versioning

2022-07-15 Thread Andrew Yourtchenko
Hi Stanislav, The api is marked as “Production” so the behavior of checkstyle is there to protect the users (as for the duplication - it is a choice to do it once in VPP or in each and every downstream consumer). As for the pure code exercise - I just did it for the sake of a test, took a grand

[vpp-dev] Facing issue with cli-listen option in vpp.conf

2022-07-15 Thread Amaresh Parida
Hi All This is regarding the cli-listen option in vpp.conf. I want to use* cli-listen localhost:5002* instead of *cli-listen /run/vpp/cli.sock* After VPP start when I execute vppctl command, I see following error in VPP *CLI# vppctl -s localhost:5002connect: No such file or directory* When I

[vpp-dev] vpp api versioning

2022-07-15 Thread Stanislav Zaikin
Hello folks, According to [0] it should be possible to add breaking changes to vpp api with incrementing the major version of the api. There's one issue in the LCP api - a C++ keyword "namespace" is used there and I want to change it to "netns" and increase a major version. But make checkstyle-api