[ACS44]release 4.4.4?

2015-05-20 Thread Daan Hoogland
H, Are there any fixes that people have that should go into 4.4? I would like to create a release candidate over the weekend.

Re: [ACS44]release 4.4.4?

2015-05-21 Thread Andrija Panic
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=173909e99d85cfcc85b017bc426950f9f16fddf0 enable CEPH image resize if possible :) On 21 May 2015 at 08:42, Daan Hoogland wrote: > H, > > Are there any fixes that people have that should go into 4.4? I would like > to create

Re: [ACS44]release 4.4.4?

2015-05-21 Thread Daan Hoogland
Andrija, can you backport and make a pull request? Op do 21 mei 2015 om 17:41 schreef Andrija Panic : > > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=173909e99d85cfcc85b017bc426950f9f16fddf0 > > enable CEPH image resize if possible :) > > On 21 May 2015 at 08:42, Daan

Re: [ACS44]release 4.4.4?

2015-05-21 Thread Andrija Panic
Daan, I'm not so sure I can pull this in short time frame - I'm no developer at all :( Any guidelines for dummies? something via github clone etc (I did that for documenation ) ? thx On 21 May 2015 at 18:11, Daan Hoogland wrote: > Andrija, can you backport and make a pull request? > > Op do 21

Re: [ACS44]release 4.4.4?

2015-05-21 Thread Remi Bergsma
Hi Andrija, Here you go: https://github.com/apache/cloudstack/pull/281 ;-) @Daan, please check if it looks OK to you. Regards, Remi On 21 May 2015, at 18:29, Andrija Panic mailto:andrija.pa...@gmail.com>> wrote: Daan, I'm not so sure I can pull this in short time frame - I'm no developer at

Re: [ACS44]release 4.4.4?

2015-05-21 Thread Andrija Panic
Thx Remi :) On May 21, 2015 11:10 PM, "Remi Bergsma" wrote: > Hi Andrija, > > Here you go: https://github.com/apache/cloudstack/pull/281 ;-) > > @Daan, please check if it looks OK to you. > > Regards, > Remi > > > On 21 May 2015, at 18:29, Andrija Panic andrija.pa...@gmail.com>> wrote: > > Daan,

Re: [ACS44]release 4.4.4?

2015-05-22 Thread Daan Hoogland
same fork -create pr procedure Op do 21 mei 2015 om 18:29 schreef Andrija Panic : > Daan, I'm not so sure I can pull this in short time frame - I'm no > developer at all :( > > Any guidelines for dummies? something via github clone etc (I did that for > documenation ) ? > > thx > > On 21 May 201

Re: [ACS44]release 4.4.4?

2015-05-22 Thread Daan Hoogland
Andrija, I am holding this one because of comments Marcus made in the pull request. Op vr 22 mei 2015 om 09:39 schreef Daan Hoogland : > same fork -create pr procedure > > Op do 21 mei 2015 om 18:29 schreef Andrija Panic >: > > Daan, I'm not so sure I can pull this in short time frame - I'm no

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Andrija Panic
Daan - I just tested this (but on 4.5.1), and it doesnt work, at least in my setup: 2015-05-28 16:14:56,108 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-1:null) Trying to fetch storage pool 8457c284-cf5d-3979-b82e-32ea5efeb97b from libvirt 2015-05-28 16:14:56,257 DEBUG [kvm.reso

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Daan Hoogland
Andrija, the code you refer to is post-4.5. it is not in 4.4 or 4.5.So I am a bit confused by your remark. How do you mean it generates the error you got? Did you apply the patch from PR-281 on 4.5.1 and tested? or did you test a vanilla 4.5.1? Op do 28 mei 2015 om 16:22 schreef Andrija Panic : >

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Andrija Panic
Ah...sorry for that, my bad... I understood it's already present on 4.5.1 vanial.. But I understood that this commit IS actually already present in 4.5.1: https://github.com/apache/cloudstack/commit/173909e99d85cfcc85b017bc426950f9f16fddf0 ? On 28 May 2015 at 16:33, Daan Hoogland wrote: > Andr

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Daan Hoogland
yes, that one is. It doesn't solve the problem, I understand. Does it worsen it? Op do 28 mei 2015 om 16:56 schreef Andrija Panic : > Ah...sorry for that, my bad... I understood it's already present on 4.5.1 > vanial.. > But I understood that this commit IS actually already present in 4.5.1: > >

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Andrija Panic
Not really, volume is still fine (undamaged) :) but resize doesnt work, seems like code-only issue, since on debug agent in logs, I dont see anything sent to libvirt etc...confirmed from inside VM. On 28 May 2015 at 16:58, Daan Hoogland wrote: > yes, that one is. It doesn't solve the problem, I

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Daan Hoogland
Marcus reported corrupted volumes due to this, in the pr: https://github.com/apache/cloudstack/pull/281#issuecomment-104432629 I am not sure it was due to this code or that it didn't get solved by this code. As it is in 4.5 and master I will leave it in for now. meaning in 4.4.4. Op do 28 mei 2015

Re: [ACS44]release 4.4.4?

2015-05-28 Thread Andrija Panic
The specific comment Marcus made was warning - but it was/is already commited in 4.5.1 vanila. But it doesnt work as it seems... Even qemu-img resize worked fine for me on poweredoff VMs/volumes mutiply times (with DB changes), and later we fix manually partitions/FS inside OS. But here also live