<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-08-03 9:53 GMT-07:00 Andrew Laski <span dir="ltr"><<a href="mailto:andrew@lascii.com" target="_blank">andrew@lascii.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>




<div><div>I think the discussion about whether or not this needs a microversion is missing the bigger question of whether or not this should be in the API to begin with. If it's safe to rollback from this error state why not just do that automatically in Nova? If it's proposed for the API because it's not considered safe I don't agree it should be in the API. This is not an API that's restricted to admins by default.<br></div>
<div><br></div>
<div>However if this is going to be exposed in the API I lean towards this not needing a microversion. It's a new policy in the usage of the API, not a change to the API. To me it seems like adding a microversion because a policy rule was changed. I know we should have some sort of signal here for users, but I think we need to look at different ways to signal this type of change.<span class="HOEnZb"><font color="#888888"><br></font></span></div></div></blockquote><div><br></div><div>Yeah, I feel a new microversion in this case seems a little overkill.<br></div><div>This is a negative case and rollback could be operated in all versions.<br></div><div>We have implemented rollback thing in years, and I guess we don't have any negative feedback related to rollback from users.<br><br></div><div>Thanks<br></div><div>Ken Omichi<br><br>---<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><span class="HOEnZb"><font color="#888888"></font></span></div><span class="HOEnZb"><font color="#888888">
<div><br></div>
<div>-Andrew<br></div></font></span><div><div class="h5">
<div><br></div>
<div><br></div>
<div>On Tue, Aug 2, 2016, at 02:11 AM, <a href="mailto:han.rong3@zte.com.cn" target="_blank">han.rong3@zte.com.cn</a> wrote:<br></div>
</div></div><blockquote type="cite"><div><div class="h5"><div><span style="font-family:sans-serif"><span style="font-size:small">patchset url: </span></span><a href="https://review.openstack.org/#/c/334747/" target="_blank"><span style="font-family:sans-serif"><span style="font-size:small">https://review.openstack.org/#/c/334747/</span></span></a> <br></div>
<div> <br></div>
<table><tbody><tr valign="top"><td bgcolor="white"><div><br></div>
<div><span style="font-size:x-small"><br></span></div>
<table align="center"><tbody><tr><td><div><br></div>
<div><span style="font-family:menlo,consolas,"courier new",monospace"><span style="font-size:small">Allow "revert_resize" to recover error instance
after resize/migrate.</span></span> <br></div>
<div> <br></div>
<div><span style="font-family:menlo,consolas,"courier new",monospace"><span style="font-size:small">When resize/migrate instance, if error occurs on source
compute node, instance state can rollback to active currently. But if error
occurs in "finish_resize" function on destination compute node,
the instance state would not rollback to active. </span></span> <br></div>
<div> <br></div>
<div><span style="font-family:menlo,consolas,"courier new",monospace"><span style="font-size:small">This patch is to rollback instance state from error
to active when resize or migrate action failed on destination compute node..</span></span><br></div>
</td></tr></tbody></table><div><br></div>
</td></tr></tbody></table><div><br></div>
<div><br></div>
<div><span style="font-family:sans-serif"><span style="font-size:small"><br></span></span><span style="font-family:menlo,consolas,"courier new",monospace"><span style="font-size:small">Best,<br> </span></span> </div>
<div><span style="font-family:menlo,consolas,"courier new",monospace"><span style="font-size:small">Rong Han</span></span> <br></div>
</div></div><span class=""><div><u>__________________________________________________________________________</u><br></div>
<div>OpenStack Development Mailing List (not for usage questions)<br></div>
<div>Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br></div>
<div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div>
</span></blockquote><div><br></div>
</div>

<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>
<br></blockquote></div><br></div></div>