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 > > > > >