Re: [Netstack] Code coverage status

2011-10-18 Thread Dan Wendlandt
Thanks for the review Edgar. Folks, since Edgar has been participating with both code + reviews, I'd like to nominate him for membership as a core dev. Please +1 if you agree. Dan On Mon, Oct 17, 2011 at 11:15 PM, Edgar Magana (eperdomo) < eperd...@cisco.com> wrote: > Hi Brad, > > Good job he

[Netstack] FW: [Openstack] Change to api-paste.ini

2011-10-18 Thread Salvatore Orlando
Is this something we might consider for Quantum as well as concerns resource extensions? Salvatore -Original Message- From: openstack-bounces+salvatore.orlando=eu.citrix@lists.launchpad.net [mailto:openstack-bounces+salvatore.orlando=eu.citrix@lists.launchpad.net] On Behalf Of

Re: [Netstack] Code coverage status

2011-10-18 Thread Brad Hall
+1 On Tue, Oct 18, 2011 at 12:07 AM, Dan Wendlandt wrote: > Thanks for the review Edgar. > Folks, since Edgar has been participating with both code + reviews, I'd like > to nominate him for membership as a core dev.  Please +1 if you agree. > Dan > > On Mon, Oct 17, 2011 at 11:15 PM, Edgar Magana

Re: [Netstack] Code coverage status

2011-10-18 Thread Salvatore Orlando
+1 > -Original Message- > From: netstack- > bounces+salvatore.orlando=eu.citrix@lists.launchpad.net > [mailto:netstack- > bounces+salvatore.orlando=eu.citrix@lists.launchpad.net] On Behalf Of > Brad Hall > Sent: 18 October 2011 16:06 > To: Dan Wendlandt > Cc: netstack@lists.launchp

Re: [Netstack] Code coverage status

2011-10-18 Thread Salvatore Orlando
Hi Brad, Thanks for this very important piece of code. I think we should also try and enforce coverage checks in Jenkins (e.g.: if coverage for a module drops below 80% reject commit). What's your opinion? Salvatore > -Original Message- > From: netstack- > bounces+salvatore.orlando=eu.

Re: [Netstack] Code coverage status

2011-10-18 Thread Brad Hall
Yes, I agree with that. I was going to email jeblair about this once the change went in. Thanks, Brad On Tue, Oct 18, 2011 at 8:30 AM, Salvatore Orlando wrote: > Hi Brad, > Thanks for this very important piece of code. > I think we should also try and enforce coverage checks in Jenkins (e.g.: i

Re: [Netstack] Code coverage status

2011-10-18 Thread Sumit Naiksatam (snaiksat)
On that note, I would also like to nominate Rohit Agarwalla for membership. He participated & will continue to do so with code and review activities. +1 for Edgar Thanks, ~Sumit. > -Original Message- > From: netstack-bounces+snaiksat=cisco@lists.launchpad.net > [mailto:netstack-boun

[Netstack] Changing the update_{network,port} calls

2011-10-18 Thread Brad Hall
Hey Everyone, Since the change to make the create() calls take a parameter of k/v pairs has gotten approval I wanted to propose a change to make the update calls do the same. The review is here: https://review.openstack.org/#change,929 .. I've posted this review in order to get feedback and to ha

Re: [Netstack] FW: [Openstack] Change to api-paste.ini

2011-10-18 Thread Brad Hall
Yeah, I think it sounds useful. Thanks, Brad On Tue, Oct 18, 2011 at 7:11 AM, Salvatore Orlando wrote: > Is this something we might consider for Quantum as well as concerns resource > extensions? > > Salvatore > > -Original Message- > From: openstack-bounces+salvatore.orlando=eu.citrix.

Re: [Netstack] Code coverage status

2011-10-18 Thread Edgar Magana (eperdomo)
+1 Rohit ...mmmhhh and myself of course :-) -Original Message- From: netstack-bounces+eperdomo=cisco@lists.launchpad.net [mailto:netstack-bounces+eperdomo=cisco@lists.launchpad.net] On Behalf Of Sumit Naiksatam (snaiksat) Sent: Tuesday, October 18, 2011 1:31 PM To: Brad Hall; D

Re: [Netstack] [Openstack] Announcing Nova subteams

2011-10-18 Thread Sumit Naiksatam (snaiksat)
Hi, Following up on the topic of nova-network parity, we also wanted to continue the discussion around how the proposed L3 service can be leveraged to achieve this. We have posted a very preliminary flow in the document here: http://wiki.openstack.org/quantum-multi-switch-plugin?action=Atta

Re: [Netstack] [Openstack] Announcing Nova subteams

2011-10-18 Thread Sumit Naiksatam (snaiksat)
Apologies for the incorrect link, the link should be: http://wiki.openstack.org/l3-service?action=AttachFile&do=view&target=qu antum-nova-parity-SumitNaiksatam-v3.pdf From: Sumit Naiksatam (snaiksat) Sent: Tuesday, October 18, 2011 2:46 PM To: 'Dan Wendlandt'; netstack@lists.launchpad.net

Re: [Netstack] Code coverage status

2011-10-18 Thread Dan Wendlandt
+1 rohit. Ok, seeing no complaints I will add both Edgar and Rohit to the core group. Welcome! dan On Tue, Oct 18, 2011 at 2:23 PM, Edgar Magana (eperdomo) wrote: > +1 Rohit > > ...mmmhhh and myself of course :-) > > -Original Message- > From: netstack-bounces+eperdomo=cisco@list

Re: [Netstack] Changing the update_{network,port} calls

2011-10-18 Thread Dan Wendlandt
On Tue, Oct 18, 2011 at 1:42 PM, Brad Hall wrote: > Hey Everyone, > > Since the change to make the create() calls take a parameter of k/v > pairs has gotten approval I wanted to propose a change to make the > update calls do the same. The review is here: > https://review.openstack.org/#change,92

Re: [Netstack] Code coverage status

2011-10-18 Thread Edgar Magana (eperdomo)
Thanks!!! From: Dan Wendlandt [mailto:d...@nicira.com] Sent: Tuesday, October 18, 2011 4:52 PM To: Edgar Magana (eperdomo) Cc: Sumit Naiksatam (snaiksat); Brad Hall; netstack@lists.launchpad.net Subject: Re: [Netstack] Code coverage status +1 rohit. Ok, seeing no complaints I will ad