<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 7, 2017 at 4:15 PM, Ghanshyam Mann <span dir="ltr"><<a href="mailto:ghanshyammann@gmail.com" target="_blank">ghanshyammann@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Thanks. I am not sure these kind of driver specific behavior on APIs<br>
side. This bring up question that should not cinder APIs be consistent<br>
from usage point. In this case[1], create backup API can accept<br>
'container' param and do/don't create pool as per configured driver?<br>
Then have better documentation for that what all driver honor that or<br>
not.<br>
<br>
Any suggestion ?<br>
<br>
..1 <a href="https://review.openstack.org/#/c/454321/3" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/454321/3</a></blockquote><div><br></div><div>Yeah, I've left a comment on that review. And another comment on <a href="https://review.openstack.org/#/c/454722">https://review.openstack.org/#/c/454722</a> :<br><br>"I'd rather we revert the change completely than to see this merged. <br><br>If the Ceph backup driver doesn't support the container argument it should either grow support for it, or ignore that argument, or we change Cinder's API completely so that the container argument is not part of the public API anymore.<br><br>Do we expect each and every user to know what each and every drivers support ? I don"t think so, so Tempest shouldn"t either."<br><br></div></div></div></div>