<div dir="ltr"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">Thanks Mike for getting me these useful reviews & design discussions.</div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif"><br></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">So as it stands now, I am trying '<i>provider_id</i>' to map OpenStack/Cinder with the driver's backend storage.</div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif"><br></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">I got some useful review comments from @xing-yang to try out '<i>provider_id'</i> feature enabled by below commit:</div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif"><a href="https://review.openstack.org/#/c/143205/">https://review.openstack.org/#/c/143205/</a><br></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif"><br></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">Do let me know if '<i>provider_id'</i> approach seems reasonable ?</div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif"><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><font style="background-color:rgb(255,255,255)" color="#000000" face="trebuchet ms, sans-serif">Regards,</font></div><font style="background-color:rgb(255,255,255)" color="#000000" face="trebuchet ms, sans-serif">Amit</font><div><a href="http://www.cloudbyte.com/" target="_blank"><font style="background-color:rgb(255,255,255)" color="#0000ff" face="trebuchet ms, sans-serif"><i>CloudByte Inc.</i></font></a></div></div></div></div>
<br><div class="gmail_quote">On Thu, Dec 25, 2014 at 1:46 AM, Mike Perez <span dir="ltr"><<a href="mailto:thingee@gmail.com" target="_blank">thingee@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 06:05 Sat 20 Dec     , Duncan Thomas wrote:<br>
> No, I mean that if drivers are going to access database, then they should<br>
> do it via a defined interface that limits what they can do to a sane set of<br>
> operations. I'd still prefer that they didn't need extra access beyond the<br>
> model update, but I don't know if that is possible.<br>
><br>
> Duncan Thomas<br>
> On Dec 19, 2014 6:43 PM, "Amit Das" <<a href="mailto:amit.das@cloudbyte.com">amit.das@cloudbyte.com</a>> wrote:<br>
><br>
> > Thanks Duncan.<br>
> > Do you mean hepler methods in the specific driver class?<br>
> > On 19 Dec 2014 14:51, "Duncan Thomas" <<a href="mailto:duncan.thomas@gmail.com">duncan.thomas@gmail.com</a>> wrote:<br>
> ><br>
> >> So our general advice has historical been 'drivers should not be<br>
> >> accessing the db directly'. I haven't had chance to look at your driver<br>
> >> code yet, I've been on vacation, but my suggestion is that if you<br>
> >> absolutely must store something in the admin metadata rather than somewhere<br>
> >> that is covered by the model update (generally provider location and<br>
> >> provider auth) then writing some helper methods that wrap the context bump<br>
> >> and db call would be better than accessing it directly from the driver.<br>
> >><br>
> >> Duncan Thomas<br>
> >> On Dec 18, 2014 11:41 PM, "Amit Das" <<a href="mailto:amit.das@cloudbyte.com">amit.das@cloudbyte.com</a>> wrote:<br>
<br>
</span>I've expressed in past reviews that we should have an interface that limits<br>
drivers access to the database [1], but received quite a bit of push<br>
back in Cinder. I recommend we stick to what has been decided, otherwise, Amit<br>
you should spend some time on reading the history of this issue [2] from<br>
previous meetings and start a rediscussion on it in the next meeting [3]. Not<br>
discouraging it, but this has been something brought up at least a couple of<br>
times now and it ends up with the same answer from the community.<br>
<br>
[1] - <a href="https://review.openstack.org/#/c/107693/14" target="_blank">https://review.openstack.org/#/c/107693/14</a><br>
[2] - <a href="http://eavesdrop.openstack.org/meetings/cinder/2014/cinder.2014-10-15-16.00.log.html#l-186" target="_blank">http://eavesdrop.openstack.org/meetings/cinder/2014/cinder.2014-10-15-16.00.log.html#l-186</a><br>
[3] - <a href="https://wiki.openstack.org/wiki/CinderMeetings" target="_blank">https://wiki.openstack.org/wiki/CinderMeetings</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Mike Perez<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>