Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Received-SPF: pass (athena.apache.org: domain of mike.tutkowski@solidfire.com
 designates 209.85.214.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <77B337AF224FD84CBF8401947098DD8715BFD4EF@SJCPEX01CL02.citrite.net>
References: 
 <CACOnxChjKJmtnBrt_38nDAgJsUB-WDKeDhYVarZNLtjUVD-5yw@mail.gmail.com>
	<77B337AF224FD84CBF8401947098DD8715BFD4EF@SJCPEX01CL02.citrite.net>
Date: Tue, 22 Oct 2013 17:41:52 -0600
Message-ID: 
 <CACOnxCiSWGQqnoZ-LKM8zLSVyS5VaU_=_QAf3sZ9djG=PLdh_w@mail.gmail.com>
Subject: Re: Question about StoragePoolHostVO
From: Mike Tutkowski <mike.tutkowski@solidfire.com>
To: Edison Su <Edison.su@citrix.com>
Cc: "dev@cloudstack.apache.org" <dev@cloudstack.apache.org>
Content-Type: multipart/alternative; boundary=001a11333a4eeedbd304e95cf188

--001a11333a4eeedbd304e95cf188
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable

Thanks, Edison.

What I was noticing is when I attach a disk to a Stopped VM on KVM and then
try to start it, the operation was failing because my data disk did not
have a row in this table tying the host ID to the storage pool ID.

I modified my SolidFireHostListener class to create such a row and now it
works.


On Tue, Oct 22, 2013 at 4:52 PM, Edison Su <Edison.su@citrix.com> wrote:

>  The storage_pool_host_ref table is usually used to build the
> relationship between storage pool and hypervisor host only. The local pat=
h
> in this table is not used by other code any more. ****
>
> If you have specific requirement, such as store a path in this table, we
> can add it DefaultHostListener, or you subclass DefaultHostListener.****
>
> HypervisorHostListener is a callback you can use, in case of host up and
> down. If you don=92t care about this events, you don=92t need to use the
> DefaultHostListener at all.****
>
> From cloudstack=92s point of view, the only requirement is that, there
> should be one entry in storage_pool_host_ref for each host and its primar=
y
> storage. As you pointed out, we should remove the entry when
> hostDisconnected is called.****
>
> *From:* Mike Tutkowski [mailto:mike.tutkowski@solidfire.com]
> *Sent:* Tuesday, October 22, 2013 12:28 PM
> *To:* dev@cloudstack.apache.org; Edison Su
> *Subject:* Question about StoragePoolHostVO****
>
> ** **
>
> Hi Edison (or anyone else viewing this :) ),****
>
> ** **
>
> I'm looking at the DefaultHostListener class you had written up a while
> ago.****
>
> ** **
>
> A couple questions:****
>
> ** **
>
> 1) In the "else" below, it does not appear we update the DB with the loca=
l
> path. Is this correct behavior (this is just a snippet of the method, of
> course, but I didn't see any DB update related to this)?****
>
> ** **
>
> 2) As you know, my storage is zone wide. Should I be adding a row to this
> table for each host that connects to the management server that's in the
> same zone as my storage? Assuming I should, should I implement the
> hostDisconnected method to remove the row? The DefaultHostListener just
> returns true for the hostDisconnected method.****
>
> ** **
>
>         StoragePoolHostVO poolHost =3D
> storagePoolHostDao.findByPoolHost(pool.getId(), hostId);****
>
>         if (poolHost =3D=3D null) {****
>
>             poolHost =3D new StoragePoolHostVO(pool.getId(), hostId,
> mspAnswer.getPoolInfo().getLocalPath()****
>
>                     .replaceAll("//", "/"));****
>
>             storagePoolHostDao.persist(poolHost);****
>
>         } else {****
>
>
> poolHost.setLocalPath(mspAnswer.getPoolInfo().getLocalPath().replaceAll("=
//",
> "/"));****
>
>         }****
>
> Thanks!
> ****
>
> ** **
>
> --
> *Mike Tutkowski*****
>
> *Senior CloudStack Developer, SolidFire Inc.*****
>
> e: mike.tutkowski@solidfire.com****
>
> o: 303.746.7302****
>
> Advancing the way the world uses the cloud<http://solidfire.com/solution/=
overview/?video=3Dplay>
> *=99*****
>


--=20
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkowski@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud<http://solidfire.com/solution/overview/?video=3Dplay>
*=99*

--001a11333a4eeedbd304e95cf188--