<div dir="ltr">Hi Andrey<br><br><div class="gmail_quote"><div dir="ltr">On Wed, 3 Aug 2016 at 14:35 Andrey Pavlov <<a href="mailto:andrey.mp@gmail.com">andrey.mp@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Instead of adding one more relation to glance, I can relate my charm to new relation 'cinder-volume-service'<br></div></blockquote><div><br></div><div>almost</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br>So there is no more changes in the glance charm.<br>And additional in my charm will be -<br><br>metadata.yaml -<br><b>subordinate: true<br>provides:<br>  image-backend:<br>    interface: cinder<br>    scope: container</b></div></blockquote><div><br></div><div>Almost - the interface type should be something like 'glance-backend' matching the type of the container scoped interface we'd need to add to the glance charm.</div><div><br></div><div>provides:</div><div>  image-backend:</div><div>    interface: glance-backend</div><div>    scope: container</div><div><br></div><div>The glance charm needs to interrogate the data presented by the subordinate charm - I'd suggest one of the data items is 'cinder-backend=True|False' in which case the glance charm can then set the required configuration option in the glance-api.conf file (instead of doing that via a relation to cinder as you proposed in your original changes to glance).</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br>and then I can relate my charm to glance (and my charm will be installed in the same container as glance, so I can configure OS)<br><b>juju add-relation glance:cinder-volume-service scaleio-openstack:image-backend</b><div><b><br></b></div><div>This option works for me - I don't need to add something to glance config. I'm only need to add files to /etc/glance/rootwrap.d/</div><div>and this option allows to do it.</div></div></blockquote><div><br></div><div>I think the experience would be something like:</div><div><br></div><div>  <b>juju add-relation glance:image-backend  <span style="line-height:1.5">scaleio-openstack:image-backend</span></b></div><div><br></div><div>based on my feedback above.  A relation to cinder might not be required at-all if all glance needs to know is 'use cinder' rather than any other additional data such as the location of the cinder-api services etc..</div><div><br></div><div>As you state - the scaleio-openstack charm can install the required filters to /etc/glance/rootwrap.d - which is great as it moves the scaleio specific bits into a specific charm for scaleio (which I like), rather than overloading and increasing the complexity of the core glance charm.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I've made additional review - <a href="https://review.openstack.org/#/c/350565/" target="_blank">https://review.openstack.org/#/c/350565/</a></div></div></blockquote><div><br></div><div>Looking today -  I think we can refine things via the review if the overall design intent is agreed here.</div><div><br></div><div>Thanks for your work on this feature!</div><div><br></div><div>James</div><div> </div></div></div>