On Tue, Oct 08, 2013 at 10:23:32AM -0400, Chip Childers wrote:
> As stated, I've imported the contrail plugin donation into the contrail
> branch.
> 
> I've taken the time to add the ASF license header to all of the new files
> in that branch.  I think we have to complete the following in order to
> merge into master.
> 
> 1) I'd like to see the package structure changed to match
> org.apache.cloudstack, instead of the Juniper namespace.  We only have
> com.cloud namespaces for legacy reasons, and are trying to consolidate into
> the apache ns.
> 
> 2) Folks with past experience with network plugins need to review the
> plugin's code and provide comments or +1s for a merge.  Chiradeep and Hugo,
> you've been "randomly" selected to help on this...  ;-)  Pedro, I'll assume
> that you will be happy to provide patches via reviewboard against this
> branch if changes are requested (including the package structure noted
> above).
> 
> 3) I'd love if we could get some consensus on what additional tests and /
> or changes to the test approach are needed.  Prasanna - as with Hugo and
> Chiradeep, you've been "randomly" selected to at least provide some input
> here.

I saw the thread earlier about a mysql db generated for performing an
integration test. If someone can point me to the spec/docs/readme on
how to run these presumably without the contrail device I'm happy to
take a look.

> 
> Anything I'm missing?
> 
> -chip

-- 
Prasanna.,

------------------------
Powered by BigRock.com

Reply via email to