<!DOCTYPE html>
<html>
<head>
<title></title>
</head>
<body><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.<br></div>
<div><br></div>
<div>-Andrew<br></div>
<div><br></div>
<div><br></div>
<div>On Tue, Aug 2, 2016, at 02:11 AM, <a href="mailto:han.rong3@zte.com.cn">han.rong3@zte.com.cn</a> wrote:<br></div>
<blockquote type="cite"><div><span style="font-family:sans-serif" class="font"><span style="font-size:small" class="size">patchset url: </span></span><a href="https://review.openstack.org/#/c/334747/"><span style="font-family:sans-serif" class="font"><span style="font-size:small" class="size">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" class="size"><br></span></div>
<table align="center"><tbody><tr><td><div><br></div>
<div><span style="font-family:menlo,consolas,"courier new",monospace" class="font"><span style="font-size:small" class="size">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" class="font"><span style="font-size:small" class="size">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" class="font"><span style="font-size:small" class="size">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" class="font"><span style="font-size:small" class="size"><br></span></span><span style="font-family:menlo,consolas,"courier new",monospace" class="font"><span style="font-size:small" class="size">Best,<br> </span></span> </div>
<div><span style="font-family:menlo,consolas,"courier new",monospace" class="font"><span style="font-size:small" class="size">Rong Han</span></span> <br></div>
<div><u>__________________________________________________________________________</u><br></div>
<div>OpenStack Development Mailing List (not for usage questions)<br></div>
<div>Unsubscribe: OpenStack-dev-request@lists.openstack.org?subject:unsubscribe<br></div>
<div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div>
</blockquote><div><br></div>
</body>
</html>