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 <daan.hoogl...@gmail.com> 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 <andrija.pa...@gmail.com
> >:
>
> > 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/%3c5917de61cbfc41a59e68f25cd7e79...@git.apache.org%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 <daan.hoogl...@gmail.com> 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 <
> > daan.hoogl...@gmail.com
> > > >:
> > >
> > > > same fork -create pr procedure
> > > >
> > > > Op do 21 mei 2015 om 18:29 schreef Andrija Panic <
> > > andrija.pa...@gmail.com
> > > > >:
> > > >
> > > > 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 <daan.hoogl...@gmail.com>
> > wrote:
> > > >>
> > > >> > Andrija, can you backport and make a pull request?
> > > >> >
> > > >> > Op do 21 mei 2015 om 17:41 schreef Andrija Panic <
> > > >> andrija.pa...@gmail.com
> > > >> > >:
> > > >> >
> > > >> > >
> > > >> > >
> > > >> >
> > > >>
> > >
> >
> 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 <daan.hoogl...@gmail.com
> >
> > > >> 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ć
