Not quite : I emailed Ashutosh about the controversial subject earlier today. We marked off 'ipclearance' as a point-in-time. This was when (January?) the tests were contributed to ASF and went into some legal issues. It is the point-in-time that is referred to in the request not the code itself. This code is post-ipclearance changes made to the framework and sent as a patch.
On Thu, Apr 25, 2013 at 12:33:07PM -0400, Chip Childers wrote: > > Did we actually process IP Clearance for these? The summary of the > review board record implies that we did. > > I do *not* see it listed on the official IP Clearance page [1]. Is this > the code that was under discussion to be granted by Citrix to ASF (but > was developed by Clogeny)? AFAIK, this was still pending CTXS signing a > new grant. If that grant *was* signed, I still have to take it through > the IP clearance process before the code is submitted. > > I'll give this 24 hours, and then revert the commit if I don't hear back > from anyone as to the legal status of this patch. > > -chip > > [1] http://incubator.apache.org/ip-clearance/index.html > > > On Wed, Apr 24, 2013 at 03:51:16PM +0000, Prasanna Santhanam wrote: > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/10741/#review19624 > > ----------------------------------------------------------- > > > > Ship it! > > > > > > Applied but edited the log message to reflect the nature of fixes. > > > > commit a1ef9d7312c2de037e09718abf367af774cc288a > > Author: Ashutosh Kelkar <ashut...@clogeny.com> > > Date: Wed Apr 24 15:03:10 2013 +0530 > > > > Multiple fixes to marvin framework > > > > 1. adding hypervisor information to configs > > TODO: support for multi-hypervisor zones? > > 2. CLOUDSTACK-601: Marvin unicode decode errors when running mysql > > queries via dbClient > > 3. adding keypair support for remotessh client > > > > Signed-off-by: Prasanna Santhanam <t...@apache.org> > > > > > > - Prasanna Santhanam > > > > > > On April 23, 2013, 9:49 p.m., Ashutosh Kelkar wrote: > > > > > > ----------------------------------------------------------- > > > This is an automatically generated e-mail. To reply, visit: > > > https://reviews.apache.org/r/10741/ > > > ----------------------------------------------------------- > > > > > > (Updated April 23, 2013, 9:49 p.m.) > > > > > > > > > Review request for cloudstack and Prasanna Santhanam. > > > > > > > > > Description > > > ------- > > > > > > Merging changes to marvin after ipclearance from cloudstack-qa > > > > > > - Base classes for Router, Tag, PrivateGateway and StaticRoute etc. > > > - VPC support for existing base classes > > > - Read hypervisor config from setting file > > > - Support for keypair authentication in remoteSSHClient > > > > > > > > > Diffs > > > ----- > > > > > > tools/marvin/marvin/asyncJobMgr.py 40304fa > > > tools/marvin/marvin/cloudstackConnection.py 214a878 > > > tools/marvin/marvin/cloudstackTestClient.py 85552ed > > > tools/marvin/marvin/dbConnection.py 8fa8643 > > > tools/marvin/marvin/deployDataCenter.py d358789 > > > tools/marvin/marvin/integration/lib/base.py 92cdf81 > > > tools/marvin/marvin/integration/lib/utils.py cff24a1 > > > tools/marvin/marvin/remoteSSHClient.py 4fb2f0d > > > > > > Diff: https://reviews.apache.org/r/10741/diff/ > > > > > > > > > Testing > > > ------- > > > > > > > > > Thanks, > > > > > > Ashutosh Kelkar > > > > > > > > -- Prasanna., ------------------------ Powered by BigRock.com