Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Dave Wallace
In that case, Billy's workaround is a valid fix. Billy, if you submit the patch and I'll merge it. Thanks, -daw- On 10/19/16 1:36 PM, Luke, Chris wrote: Ahh, yes. Agree. *From:*Burt Silverman *Sent:* 10/19/16, 12:33 PM *To:* Luke, Chris *Cc:* Billy McFall, vpp-dev *Subject:* Re: [vpp-dev] P

Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Luke, Chris
Ahh, yes. Agree. From: Burt Silverman Sent: 10/19/16, 12:33 PM To: Luke, Chris Cc: Billy McFall, vpp-dev Subject: Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM Now I understand; this is "sudo -u vagrant" so it is not trying to become root. So it should be fine (I presume), and yes, i

Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Burt Silverman
Now I understand; this is "sudo -u vagrant" so it is not trying to become root. So it should be fine (I presume), and yes, it (SUDCOMD) has to be used consistently, across the board. Burt On Wed, Oct 19, 2016 at 10:13 AM, Luke, Chris wrote: > Though in general less sudo is desirable than more;

Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Dave Wallace
Billy, This is the same failure that I am seeing on MacOsX 10.11.6/Vagrant 1.8.6/Virtualbox Version 5.1.6 r110634. Thanks for taking the time to reproduce/diagnose the issue. -daw- On 10/19/16 8:43 AM, Billy McFall wrote: In the VPP Meeting yesterday (10/18/2016), Dave Wallace reported havi

Re: [vpp-dev] Can't build Centos Vagrant Master

2016-10-19 Thread Calvin Ference
I've actually been manually copying my keys from my host into the vagrant guest, that seems to solve the permission denied issue. Probably not the most secure thing to do, but it works. But something did change in that aspect as this wasn't an issue previously. On 18 October 2016 at 22:09, Burt S

Re: [vpp-dev] [csit-dev] DO_NOT_MERGE

2016-10-19 Thread Dave Wallace
+1 On 10/19/16 8:11 AM, Damjan Marion (damarion) wrote: I think simply allowing “Change Owner” to do -2 on his own change should address all issues here as only person who put -2 can remove it. (you remember that i had to ping you to remove -2 on one change as I was not able to merge it…)

Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Burt Silverman
Agree, absolutely. Someone might take a look globally why $SUDOCMD was needed in the first place. But I am seeing that it is basically used throughout the entire build.sh file, designed in, and the anomaly was the later addition of that one line (in red) that does not use SUDOCMD. So perhaps author

Re: [vpp-dev] vpp-csit-verify-virl-master failing on vpp grerrit patches

2016-10-19 Thread Dave Wallace
John, It appears that this was due to a connectivity issue which has now magically resolved itself. Thanks, -daw- On 10/18/16 4:28 PM, John Lo (loj) wrote: It seems vpp-csit-verify-virl-master has been failing for recently submitted vpp patches. I was looking at this one: https://gerrit

Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Luke, Chris
Though in general less sudo is desirable than more; this is indicative of too much use of sudo. Chris. From: vpp-dev-boun...@lists.fd.io [mailto:vpp-dev-boun...@lists.fd.io] On Behalf Of Burt Silverman Sent: Wednesday, October 19, 2016 9:10 AM To: Billy McFall Cc: vpp-dev Subject: Re: [vpp-de

Re: [vpp-dev] [csit-dev] DO_NOT_MERGE

2016-10-19 Thread Dave Barach (dbarach)
Sounds like a good idea to me... Thanks… Dave From: vpp-dev-boun...@lists.fd.io [mailto:vpp-dev-boun...@lists.fd.io] On Behalf Of Maciek Konstantynowicz (mkonstan) Sent: Wednesday, October 19, 2016 8:39 AM To: Damjan Marion (damarion) Cc: csit-...@lists.fd.io; vpp-dev Subject: Re: [vpp-dev] [cs

Re: [vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Burt Silverman
Parallelism is always nice. Looking at your snippet from git diff, I would say it makes sense to me that if Ubuntu requires SUDOCMD, then so does CentOS. Just my two cent(O)s. Burt On Wed, Oct 19, 2016 at 8:43 AM, Billy McFall wrote: > In the VPP Meeting yesterday (10/18/2016), Dave Wallace re

[vpp-dev] Permissions Error on Centos Vagrant VPP VM

2016-10-19 Thread Billy McFall
In the VPP Meeting yesterday (10/18/2016), Dave Wallace reported having issues compiling VPP in a Vagrant VM running Centos. The thought was it might be a MAC issue. I was able to reproduce on a Fedora 24 host. Dave, let me know if this is not the same issue you were seeing. I created Jira https://

Re: [vpp-dev] [csit-dev] DO_NOT_MERGE

2016-10-19 Thread Maciek Konstantynowicz (mkonstan)
+1 -Maciek On 19 Oct 2016, at 13:11, Damjan Marion (damarion) mailto:damar...@cisco.com>> wrote: I think simply allowing “Change Owner” to do -2 on his own change should address all issues here as only person who put -2 can remove it. (you remember that i had to ping you to remove -2 on one

Re: [vpp-dev] [csit-dev] DO_NOT_MERGE

2016-10-19 Thread Maciek Konstantynowicz (mkonstan)
+1 -Maciek On 19 Oct 2016, at 13:09, Damjan Marion (damarion) mailto:damar...@cisco.com>> wrote: Inline… On 19 Oct 2016, at 08:20, Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES at Cisco) mailto:pmi...@cisco.com>> wrote: Hello, This is more about democracy/autocracy rather than mechanics of

Re: [vpp-dev] [csit-dev] DO_NOT_MERGE

2016-10-19 Thread Damjan Marion (damarion)
I think simply allowing “Change Owner” to do -2 on his own change should address all issues here as only person who put -2 can remove it. (you remember that i had to ping you to remove -2 on one change as I was not able to merge it…) On 18 Oct 2016, at 20:32, Keith Burns mailto:alaga...@gmail

Re: [vpp-dev] [csit-dev] DO_NOT_MERGE

2016-10-19 Thread Damjan Marion (damarion)
Inline… On 19 Oct 2016, at 08:20, Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES at Cisco) mailto:pmi...@cisco.com>> wrote: Hello, This is more about democracy/autocracy rather than mechanics of gerrit. I’ve put -1 (with comments) in past and it was merged anyway by someone who put +2 ☺. As c

Re: [vpp-dev] [csit-dev] Hugepages saga .. back in game ?

2016-10-19 Thread Damjan Marion
Because tests were failing... > On 19 Oct 2016, at 13:43, Maciek Konstantynowicz (mkonstan) > wrote: > > +vpp-dev, csit-dev > > Folks, Can anybody in the know look at Peter’s points below ? > Looks like we’re stuck in the csit-vpp hugepages saga again :( > > -Maciek > >> On 19 Oct 2016, at

Re: [vpp-dev] flowtable proposal

2016-10-19 Thread gannega
Hi Ray, I put the scripts in ft-test at the root of the repository. I cloned vpp completely and chose to present the plugins in independent commits, followed by the tests (so, 3 commits). Best regards, On 10/19/16 13:35, Kinsella, Ray wrote: > Gabriel, > > Where do I find the scripts? > > Deli

Re: [vpp-dev] Hugepages saga .. back in game ?

2016-10-19 Thread Maciek Konstantynowicz (mkonstan)
+vpp-dev, csit-dev Folks, Can anybody in the know look at Peter’s points below ? Looks like we’re stuck in the csit-vpp hugepages saga again :( -Maciek On 19 Oct 2016, at 06:35, Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES at Cisco) mailto:pmi...@cisco.com>> wrote: This will return us back o

Re: [vpp-dev] flowtable proposal

2016-10-19 Thread Kinsella, Ray
Gabriel, Where do I find the scripts? Delighted to see contributions, but am a bit fuzzy on where I might use this. Ray K On 19/10/2016 11:11, gannega wrote: Hi, Following Christophe Fontaine's work during ietf hackathon in Berlin (https://github.com/christophefontaine/flowtable-plugin),

Re: [vpp-dev] question about VPP threading

2016-10-19 Thread Neale Ranns
Hi Madhi, have a read of; vlib/vlib/vlib_process_doc.h that might be what you want. regards, neale On 19/10/16 09:52, mahdi akrami wrote: Hi all, I studied multi-threading in VPP. As I understood we can define and use threads just for handling NICs (queues). How can I define a thread for d

Re: [vpp-dev] Is there a way to change the MAC address in VPP 1609?

2016-10-19 Thread Kinsella, Ray
Hi Chengwei, Are you using DPDK? as many pmds do support this. Not sure it's wired into VPP though? typedef void (*eth_mac_addr_remove_t)(struct rte_eth_dev *dev, uint32_t index); typedef void (*eth_mac_addr_add_t)(struct rte_eth_dev *dev, struct ether_addr *m

[vpp-dev] flowtable proposal

2016-10-19 Thread gannega
Hi, Following Christophe Fontaine's work during ietf hackathon in Berlin (https://github.com/christophefontaine/flowtable-plugin), I wrote a flowtable as a vpp plugin. Code is available here: https://github.com/GabrielGanne/vpp-flowtable/commits/master It contains the flowtable plugin, a port-mi

Re: [vpp-dev] The latest master code build failure

2016-10-19 Thread gannega
Hi, This behavior follows this commit : 4c42e913c4f9723d27f6501f6e1b6a13b9c2f9f8 You should *make build-vpp-api* before building plugins. Regards, On 10/19/16 11:14, yu scott wrote: > Hi all > > I build the stable/1609 stable version code successfully in my pc,but > when I update my > vpp code

[vpp-dev] The latest master code build failure

2016-10-19 Thread yu scott
Hi all I build the stable/1609 stable version code successfully in my pc,but when I update my vpp code to the lastest version,exec "make build" successfully, make plugins failure the error message is : ---

[vpp-dev] question about VPP threading

2016-10-19 Thread mahdi akrami
Hi all, I studied multi-threading in VPP. As I understood we can define and use threads just for handling NICs (queues). How can I define a thread for doing other jobs periodically on shared memory created by other typical VPP threads? Please reply! Thanks Mahdi __