<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:#000000"><br></div><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 3, 2017 at 11:12 AM, Zhenyu Zheng <span dir="ltr"><<a href="mailto:zhengzhenyulixi@gmail.com" target="_blank">zhengzhenyulixi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi All,<div><br></div><div><div>Horizon provided the feature to allow user edit existing flavors, no matter it is currently used by any instances or not. While Nova doesn't provide this kind of ability, Horizon achieved this by deleting the old flavor first and create a new flavor with the requested properties. And the flavor ID will be changed to a new UUID.</div><div><br></div><div>This causes problem when display instances with latest Nova, Horizon display flavor name by request flavor details using the flavor id returned by get_instance request, because Nova now moved flavor to api_db, and when a flavor got deleted, it will be directly removed from the DB, and when displaying, the field will be "Not Available".</div></div><div><br></div><div>Should we stop support editing existing flavors? It is actually deleting-and-create.</div></div></blockquote><div><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)">​I do not know actual use case if this feature​ but i feel like same as you to stop supporting this. After microversion 2.47, it would not be "Not Available' but it will still show the flavor info where that flavor does not exit in system anymore.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)">If someone refer the existing instance to use the same flavor info for their new instance the will be confused about no such flavor exist.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)">Other solution might be to allow only non used flavor but that need to iterate all existing instance and check is anyone use that flavor or not which is more difficult after 2.47 where nova return flavor detail instead of flavor id.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Maybe we should  at least add WARNING notes about  this when editing flavors, about how it is actually done and what kind of influence will be to the existing instances. </div><div><br></div><div>Nova now(> microversion 2.47) can reply embedded flavor details including ram, vcpus, original_name etc.</div><div><br></div><div>Since we provided this flavor editing feature and we displayed "name" as a identifier, after we done some flavor edit, even we fix the above mentioned problem and displayed the correct flavor name, the flavor details will be different than the actual instance type.</div></div></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)">​This also can lead to same issue of having stale flavor info in instance. ​</div><br></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><br></div><div>Thoughts?</div><div><br></div><div>BR,</div><div><br></div><div>Kevin Zheng</div><div><span style="font-size:12pt;font-family:宋体"><img width="0" height="0" border="0" style="margin-right:0px"></span></div></div></blockquote><div><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)">​-gmann</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)">​</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><span style="font-size:12pt;font-family:宋体"></span><br></div></div>
<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>
<br></blockquote></div><br></div></div>