Re: Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-25 Thread Prasanna Santhanam
Sorry - failed to push this last night. Here's the actual commit: commit 2f40a90c53067c3901cf9e5c9a33309907e48aab Author: Ashutosh Kelkar Date: Wed Apr 24 15:03:10 2013 +0530 Multiple fixes to marvin framework 1. adding hypervisor information to configs TODO: support for m

Re: Potential problem with IP ownership of a commit. WAS: Re: Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-25 Thread Chip Childers
On Thu, Apr 25, 2013 at 10:25:12PM +0530, Prasanna Santhanam wrote: > 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

Re: Potential problem with IP ownership of a commit. WAS: Re: Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-25 Thread Prasanna Santhanam
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. T

Potential problem with IP ownership of a commit. WAS: Re: Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-25 Thread Chip Childers
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

Re: Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-24 Thread Prasanna Santhanam
--- 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 o

Re: Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-23 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10741/#review19618 --- tools/marvin/marvin/integration/lib/base.py

Review Request: Merging changes to marvin after ipclearance from cloudstack-qa

2013-04-23 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10741/ --- Review request for cloudstack and Prasanna Santhanam. Description --- Merg