<br><br><div class="gmail_quote">On Wed, Oct 24, 2012 at 3:20 PM, Jonathan Proulx <span dir="ltr"><<a href="mailto:jon@jonproulx.com" target="_blank">jon@jonproulx.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On Wed, Oct 24, 2012 at 3:01 PM, John Griffith<br>
<<a href="mailto:john.griffith@solidfire.com" target="_blank">john.griffith@solidfire.com</a>> wrote:<br>
<br>
> Hey Jon,<br>
><br>
> Couple of things going on, one is the volume naming (in progress here:<br>
> <a href="https://review.openstack.org/#/c/14615/" target="_blank">https://review.openstack.org/#/c/14615/</a>).  I'll take a closer look at some<br>
> of the other issues you pointed out.<br>
<br>
</div>Hi John,<br>
<br>
On this issue I think the issue you link to above covers my problem<br>
(even if that exact implementation now seems abandoned).  So "the<br>
right thing" is to normalize all the naming on UUID in my case using<br>
lvrename (or symlinks as the patchset above does) and updating the<br>
cinder db provider_location.  What other issues did you see to look<br>
into?  Seems this covers everything I brought up here and the issues<br>
you helped me with on IRC earlier (thanks) seem like documentation<br>
issues (which I hope to get into the docs once I clean up my notes)<br>
<br>
-Jon<br>
</blockquote></div>Hey Jon,<div><br></div><div>Cool... Yeah, I had intended for that patch to be a stable/folsom patch but shouldn't have submitted it to master :(  The real problem isn't just normalizing the lvm names, but also the provider_location information that is stored in the DB for when you try to attach to your compute node.</div>

<div><br></div><div>The cinder version is the same (<a href="https://review.openstack.org/#/c/14790/">https://review.openstack.org/#/c/14790/</a>) and depending on the feedback it's a candidate for back-port.</div>
<div><br></div><div>The other possible issues that I've seen people run in to:</div><div>1. volumes directory not being specified correctly</div><div>2. not restarting tgtadm</div><div>3. not having the include statement in /etc/tgt/conf.d</div>
<div><br></div><div>I think you and I covered most of these in our chat on IRC earlier today...</div>
<div><br></div><div>One other thing that Vish pointed out is I made an assumption about attached volumes that may result in you having to detach/reattach after the upgrade.  I'm looking into that one now.</div><div><br>

</div><div>Thanks,</div><div>John</div>