Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Received-SPF: pass (nike.apache.org: local policy includes SPF record at
 spf.trusted-forwarder.org)
MIME-Version: 1.0
Date: Tue, 8 Oct 2013 10:23:32 -0400
Message-ID: 
 <CA+96GG51osjrex0RzkjACSWG1f2J_k7QMNZhdXp5_jAUFZNkOA@mail.gmail.com>
Subject: Contrail plugin
From: Chip Childers <chip.childers@sungard.com>
To: "<dev@cloudstack.apache.org>" <dev@cloudstack.apache.org>
Cc: Pedro Marques <pedro.r.marques@gmail.com>,
	Chiradeep Vittal <Chiradeep.Vittal@citrix.com>,
	Hugo Trippaers <HTrippaers@schubergphilis.com>,
	Prasanna Santhanam <prasanna.santhanam@citrix.com>
Content-Type: multipart/alternative; boundary=001a11c366946a12a304e83b8373

--001a11c366946a12a304e83b8373
Content-Type: text/plain; charset=ISO-8859-1

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.

Anything I'm missing?

-chip

--001a11c366946a12a304e83b8373--