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 <[email protected]> wrote: > 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 <[email protected] > >: > > > 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.resource.LibvirtComputingResource] > > (agentRequest-Handler-1:null) Failed to resize volume: Cannot determine > > resize type from pool type RBD > > 2015-05-28 16:14:56,257 DEBUG [cloud.agent.Agent] > > (agentRequest-Handler-1:null) Seq 1-4646870390516222369: { Ans: , > MgmtId: > > 90520740254323, via: 1, Ver: v1, Flags: 10, > > > > > [{"com.cloud.agent.api.storage.ResizeVolumeAnswer":{"newSize":0,"result":false,"details":"Failed > > to resize volume: Cannot determine resize type from pool type > > RBD","wait":0}}] } > > > > This is from agent logs, and same messages in management logs. > > No logs in libvirt.log > > > > Found some interesting code here, that generates the error I got - > > > > > http://mail-archives.apache.org/mod_mbox/cloudstack-commits/201505.mbox/%[email protected]%3E > > We are using Ubuntu 14.04, ACS 4.5.1, defautl versions of libvirt etc... > > > > Regards, > > Andrija > > > > > > On 22 May 2015 at 09:42, Daan Hoogland <[email protected]> wrote: > > > > > 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 < > > [email protected] > > > >: > > > > > > > same fork -create pr procedure > > > > > > > > Op do 21 mei 2015 om 18:29 schreef Andrija Panic < > > > [email protected] > > > > >: > > > > > > > > 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 <[email protected]> > > wrote: > > > >> > > > >> > Andrija, can you backport and make a pull request? > > > >> > > > > >> > Op do 21 mei 2015 om 17:41 schreef Andrija Panic < > > > >> [email protected] > > > >> > >: > > > >> > > > > >> > > > > > >> > > > > > >> > > > > >> > > > > > > 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 <[email protected] > > > > > >> wrote: > > > >> > > > > > >> > > > 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. > > > >> > > > > > > >> > > > > > >> > > > > > >> > > > > > >> > > -- > > > >> > > > > > >> > > Andrija Panić > > > >> > > > > > >> > > > > >> > > > >> > > > >> > > > >> -- > > > >> > > > >> Andrija Panić > > > >> > > > > > > > > > > > > > > > -- > > > > Andrija Panić > > > -- Andrija Panić
