<div dir="ltr">I was thinking, the current "edit" in Horizon is delete-and-create, and it is there maybe just because<div>flavor has many fields, user may want to have a new flavor but just modify one of the old flavor, so</div><div>they don't want to manually copy all other fields. And it is the automatic delete action that causes</div><div>all the problem. Maybe horizon can provide a copy-and-modify action and leave the deletion of</div><div>the flavor to the admin.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 3, 2017 at 5:59 PM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</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 08/02/2017 10:12 PM, Zhenyu Zheng wrote:<br>
> Hi All,<br>
><br>
> Horizon provided the feature to allow user edit existing flavors, no<br>
> matter it is currently used by any instances or not. While Nova doesn't<br>
> provide this kind of ability, Horizon achieved this by deleting the old<br>
> flavor first and create a new flavor with the requested properties. And<br>
> the flavor ID will be changed to a new UUID.<br>
><br>
> This causes problem when display instances with latest Nova, Horizon<br>
> display flavor name by request flavor details using the flavor id<br>
> returned by get_instance request, because Nova now moved flavor to<br>
> api_db, and when a flavor got deleted, it will be directly removed from<br>
> the DB, and when displaying, the field will be "Not Available".<br>
><br>
> Should we stop support editing existing flavors? It is actually<br>
> deleting-and-create.<br>
<br>
</span>Yes, it should not be a feature in Horizon, because it's extremely<br>
misleading what it does. It's actually really breaking their<br>
environment. It's unfortunate that it was implemented there at all. :(<br>
<span class=""><br>
> Maybe we should  at least add WARNING notes about  this when editing<br>
> flavors, about how it is actually done and what kind of influence will<br>
> be to the existing instances.<br>
><br>
> Nova now(> microversion 2.47) can reply embedded flavor details<br>
> including ram, vcpus, original_name etc.<br>
><br>
> Since we provided this flavor editing feature and we displayed "name" as<br>
> a identifier, after we done some flavor edit, even we fix the above<br>
> mentioned problem and displayed the correct flavor name, the flavor<br>
> details will be different than the actual instance type.<br>
<br>
</span>There was an extremely long an heated discussion in the Nova room in<br>
Atlanta about including that name in server document because of this<br>
Horizon behavior. I came down on the side of showing it, because people<br>
that use the flavor "rename" function are actually basically breaking<br>
their environment (in many ways). Lots of people don't do that, so this<br>
is useful to them so that they can create new instances like the ones there.<br>
<br>
The only way to change the flavor of an instance is to resize it.<br>
<span class="HOEnZb"><font color="#888888"><br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
<a href="http://dague.net" rel="noreferrer" target="_blank">http://dague.net</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br></div>