RE: re-implement clvm

2012-08-08 Thread Edison Su
r.apache.org > Subject: Re: re-implement clvm > > I'm at home, don't have access to the copy I cloned yesterday, but > wouldn't this have been in there being that the date is May 24? Or was > it committed in a branch that was merged (sorry, not super versed in > gi

Re: re-implement clvm

2012-08-07 Thread Marcus Sorensen
iso to > hypervisor host, then stop/start ssvm. > >> -Original Message- >> From: Marcus Sorensen [mailto:shadow...@gmail.com] >> Sent: Tuesday, August 07, 2012 3:57 PM >> To: cloudstack-dev@incubator.apache.org >> Subject: Re: re-implement clvm &

RE: re-implement clvm

2012-08-07 Thread Edison Su
, 2012 3:57 PM > To: cloudstack-dev@incubator.apache.org > Subject: Re: re-implement clvm > > hacked a fix by editing /var/cache/cloud/cmdline in the ssvm to use > NfsSecondaryStorageResource. Now I hit > "http://bugs.cloudstack.org/browse/CS-15143";, x != java.lang.Str

Re: re-implement clvm

2012-08-07 Thread Marcus Sorensen
; using equivalent dmsetup commands. At >> that point we had something that looked the same as the original >> patch, with only minor changes in syntax and variable names. >> >> On Thu, Aug 2, 2012 at 5:11 PM, Edison Su wrote: >>> >>> >>>>

Re: re-implement clvm

2012-08-07 Thread Marcus Sorensen
h, with only minor changes in syntax and variable names. > > On Thu, Aug 2, 2012 at 5:11 PM, Edison Su wrote: >> >> >>> -Original Message- >>> From: Marcus Sorensen [mailto:shadow...@gmail.com] >>> Sent: Thursday, August 02, 2012 3:09 PM >>

Re: re-implement clvm

2012-08-06 Thread Marcus Sorensen
02, 2012 3:09 PM >> To: cloudstack-dev@incubator.apache.org >> Subject: RE: re-implement clvm >> >> Oh, I understand, but most of the relevant code is implemented in >> 'if/else >> if/else' blocks, such that its simply a matter of copying the existing &g

RE: re-implement clvm

2012-08-02 Thread Edison Su
> -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Thursday, August 02, 2012 3:09 PM > To: cloudstack-dev@incubator.apache.org > Subject: RE: re-implement clvm > > Oh, I understand, but most of the relevant code is implemented in

RE: re-implement clvm

2012-08-02 Thread Marcus Sorensen
> code, that's just what it is. > > I'm not a lawyer so please don't take this as legal advice from Citrix or > me. > > -kevin > > > > -Original Message- > > From: Marcus Sorensen [mailto:shadow...@gmail.com] > > Sent: Tuesday,

RE: re-implement clvm

2012-08-02 Thread Kevin Kluge
or.apache.org > Subject: Re: re-implement clvm > > Here's the refactored patch. The CLVM stuff is basically a copy of the RBD > additions; and the patch also includes the original changes to > managesnapshot.sh, which is unmodified. > > I'm personally more concer

Re: re-implement clvm

2012-07-31 Thread David Nalley
On Tue, Jul 31, 2012 at 6:08 PM, Marcus Sorensen wrote: > Here's the refactored patch. The CLVM stuff is basically a copy of the > RBD additions; and the patch also includes the original changes to > managesnapshot.sh, which is unmodified. > > I'm personally more concerned about the core functiona

Re: re-implement clvm

2012-07-31 Thread David Nalley
On Tue, Jul 31, 2012 at 7:30 PM, Marcus Sorensen wrote: > sure, just let me know how to send it in. Go to reviews.apache.org Create an account Submit the patch - add the 'group' cloudstack You should see it appear on the mailing list shortly thereafter. --David

Re: re-implement clvm

2012-07-31 Thread Marcus Sorensen
y 31, 2012 3:09 PM >> To: cloudstack-dev@incubator.apache.org >> Subject: Re: re-implement clvm >> >> Here's the refactored patch. The CLVM stuff is basically a copy of the >> RBD additions; and the patch also includes the original changes to >> managesnap

RE: re-implement clvm

2012-07-31 Thread Edison Su
Could you send the patch to reviewboard, and remove the snapshot part? > -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Tuesday, July 31, 2012 3:09 PM > To: cloudstack-dev@incubator.apache.org > Subject: Re: re-implement clvm > >

Re: re-implement clvm

2012-07-31 Thread Marcus Sorensen
ent: Tuesday, July 31, 2012 1:33 PM >> To: cloudstack-dev@incubator.apache.org >> Subject: Re: re-implement clvm >> >> Ok, so I've created a refactored patch that seems to work. It was >> pretty much entirely the RBD additions that were blocking the original >> from being

RE: re-implement clvm

2012-07-31 Thread Edison Su
ubator.apache.org > Subject: Re: re-implement clvm > > Ok, so I've created a refactored patch that seems to work. It was > pretty much entirely the RBD additions that were blocking the original > from being rolled back in. If a developer would be willing to take on > the whole

Re: re-implement clvm

2012-07-31 Thread Marcus Sorensen
Ok, so I've created a refactored patch that seems to work. It was pretty much entirely the RBD additions that were blocking the original from being rolled back in. If a developer would be willing to take on the whole license issue and see this functionality put back in I'd still be willing to pay h

Re: re-implement clvm

2012-07-31 Thread Wido den Hollander
On 07/31/2012 10:12 PM, David Nalley wrote: On Tue, Jul 31, 2012 at 3:56 PM, Wido den Hollander wrote: On 07/31/2012 09:48 PM, Marcus Sorensen wrote: I'd be happy to try more if I had access to any contact info. As it is, things in the surrounding code have changed enough that a bit of r

Re: re-implement clvm

2012-07-31 Thread David Nalley
On Tue, Jul 31, 2012 at 3:56 PM, Wido den Hollander wrote: > > > On 07/31/2012 09:48 PM, Marcus Sorensen wrote: >> >> I'd be happy to try more if I had access to any contact info. As it >> is, things in the surrounding code have changed enough that a bit of >> re-factoring would need to be done e

Re: re-implement clvm

2012-07-31 Thread Marcus Sorensen
That's my problem too. I'm about 75% of the way through hacking my own patch together, and it looks at least superficially like most of the RBD code was based on the CLVM stuff (extra if statements added and CLVM replaced with RBD). I'm comfortable with reusing this in my environment, but getting i

Re: re-implement clvm

2012-07-31 Thread Wido den Hollander
On 07/31/2012 09:48 PM, Marcus Sorensen wrote: I'd be happy to try more if I had access to any contact info. As it is, things in the surrounding code have changed enough that a bit of re-factoring would need to be done even if there were permission. My hunch is that unless he's switched roles

Re: re-implement clvm

2012-07-31 Thread Marcus Sorensen
I'd be happy to try more if I had access to any contact info. As it is, things in the surrounding code have changed enough that a bit of re-factoring would need to be done even if there were permission. My hunch is that unless he's switched roles, once the new version is released he may come out

Re: re-implement clvm

2012-07-31 Thread David Nalley
On Tue, Jul 31, 2012 at 3:05 PM, Simon Weller wrote: > I recall David Nalley collecting a Cloudstack/cloud.com ICLA from rommer for > this contribution in Q4 of last year. > > > David, > > > Does the ICLA contain any contact information not available on the old > cloudstack-devel list? > > > - S

Re: re-implement clvm

2012-07-31 Thread Simon Weller
I recall David Nalley collecting a Cloudstack/cloud.com ICLA from rommer for this contribution in Q4 of last year. David, Does the ICLA contain any contact information not available on the old cloudstack-devel list? - Si - Original Message - From: "Marcus Sorensen" To: cloud