<div dir="ltr"><div dir="ltr">I haven't implemented any checks, but I did take a shot at laying down the scaffolding for implementing upgrade checks in glance [0].<div><br></div><div>Anyone who is more familiar with glance should be able to build off of that commit by implementing specific checks in glance/cmd/status.py</div><div><br></div><div>[0] <a href="https://review.openstack.org/#/c/610661/">https://review.openstack.org/#/c/610661/</a></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 15, 2018 at 10:49 AM Brian Rosmaita <<a href="mailto:rosmaita.fossdev@gmail.com">rosmaita.fossdev@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 9/24/18 3:13 PM, Matt Riedemann wrote:<br>
> On 9/24/2018 2:06 PM, Matt Riedemann wrote:<br>
>> Are there more specific docs about how to configure the 'image import'<br>
>> feature so that I can be sure I'm careful? In other words, are there<br>
>> specific things a "glance-status upgrade check" check could look at<br>
>> and say, "your image import configuration is broken, here are details<br>
>> on how you should do this"<br>
Apologies for this delayed reply.<br>
> I guess this answers the question about docs:<br>
> <br>
> <a href="https://docs.openstack.org/glance/latest/admin/interoperable-image-import.html" rel="noreferrer" target="_blank">https://docs.openstack.org/glance/latest/admin/interoperable-image-import.html</a><br>
<br>
Yes, you found the correct docs.  They could probably use a revision to<br>
eliminate some of the references to Pike and Queens, but I think the<br>
content is accurate with respect to proper configuration of image import.<br>
> Would a basic upgrade check be such that if glance-api.conf contains<br>
> enable_image_import=False, you're going to have issues since that option<br>
> is removed in Rocky?<br>
<br>
I completely missed this question when I saw this email a few weeks ago.<br>
<br>
Yes, if a Queens glance-api.conf has enable_image_import=False, then it<br>
was disabled on purpose since the default in Queens was True.  Given the<br>
Rocky defaults for import-related config (namely, all import_methods are<br>
enabled), the operator may need to make some kind of adjustment.<br>
<br>
As a side point, although the web-download import method is enabled by<br>
default in Rocky, it has whitelist/blacklist configurability to restrict<br>
what kind of URIs end-users may access.  By default, end users are only<br>
able to access URIs using the http or https scheme on the standard ports.<br>
<br>
Thanks for working on the upgrade-checker goal for Glance!<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>