<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 25, 2020 at 12:39 PM Harald Jensas <<a href="mailto:hjensas@redhat.com">hjensas@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 8/20/20 5:49 PM, Julia Kreger wrote:<br>
> I suspect we're going to have lots of push back from the TripleO<br>
> community because there has been resistance to change their default<br>
> usage in the past. As such I'm adding them to the subject so hopefully<br>
> they will be at least aware.<br>
<br>
Since TripleO already support using the direct interface, it's <br>
recommended and tested by the TripleO group focusing on edge type <br>
deployments, switching to direct by default might not be too much of a <br>
hassle for TripleO.<br></blockquote><div><br></div><div><a class="gmail_plusreply" id="plusReplyChip-0">++</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
We may want to change the disk-image format used by TripleO to raw as <br>
well, to benefit from the raw image streaming capabilities? Or would <br>
enabling image_download_source = http convert the images as they are <br>
cached on conductors? (see question inline below.)<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
> On Thu, Aug 20, 2020 at 1:57 AM Dmitry Tantsur <<a href="mailto:dtantsur@redhat.com" target="_blank">dtantsur@redhat.com</a>> wrote:<br>
>><br>
>> Hi all,<br>
>><br>
>> Side note for those lacking context: this proposal concerns deprecating one of the ironic deploy interfaces detailed in <a href="https://docs.openstack.org/ironic/latest/admin/interfaces/deploy.html" rel="noreferrer" target="_blank">https://docs.openstack.org/ironic/latest/admin/interfaces/deploy.html</a>. It does not affect the boot-from-iSCSI feature.<br>
>><br>
>> I would like to propose deprecating and removing the 'iscsi' deploy interface over the course of the next 2 cycles. The reasons are:<br>
>> 1) The iSCSI deploy is a source of occasional cryptic bugs when a target cannot be discovered or mounted properly.<br>
>> 2) Its security is questionable: I don't think we even use authentication.<br>
>> 3) Operators confusion: right now we default to the iSCSI deploy but pretty much direct everyone who cares about scalability or security to the 'direct' deploy.<br>
>> 4) Cost of maintenance: our feature set is growing, our team - not so much. iscsi_deploy.py is 800 lines of code that can be removed, and some dependencies that can be dropped as well.<br>
>><br>
>> As far as I can remember, we've kept the iSCSI deploy for two reasons:<br>
>> 1) The direct deploy used to require Glance with Swift backend. The recently added [agent]image_download_source option allows caching and serving images via the ironic's HTTP server, eliminating this problem. I guess we'll have to switch to 'http' by default for this option to keep the out-of-box experience.<br>
>> 2) Memory footprint of the direct deploy. With the raw images streaming we no longer have to cache the downloaded images in the agent memory, removing this problem as well (I'm not even sure how much of a problem it is in 2020, even my phone has 4GiB of RAM).<br>
>><br>
<br>
When using image_download_source = http, does Ironic convert non-raw <br>
images when they are placed on each conductors cache? To benefit from <br>
the raw image streaming?<br></blockquote><div><br></div><div>Yes, unless it's explicitly disabled. Although storing raw images from the beginning may make deployments a bit faster and save some disk space for this conversion.</div><div><br></div><div>Dmitry<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>> If this proposal is accepted, I suggest to execute it as follows:<br>
>> Victoria release:<br>
>> 1) Put an early deprecation warning in the release notes.<br>
>> 2) Announce the future change of the default value for [agent]image_download_source.<br>
>> W release:<br>
>> 3) Change [agent]image_download_source to 'http' by default.<br>
>> 4) Remove iscsi from the default enabled_deploy_interfaces and move it to the back of the supported list (effectively making direct deploy the default).<br>
>> X release:<br>
>> 5) Remove the iscsi deploy code from both ironic and IPA.<br>
>><br>
>> Thoughts, opinions, suggestions?<br>
>><br>
>> Dmitry<br>
> <br>
<br>
<br>
</blockquote></div></div>