<div dir="ltr"><div>It's too difficult to understand your <span style="color:rgb(51,51,51);font-family:sans-serif;font-size:12px;line-height:18px">interoperability</span> perspective.</div><div><br></div>I think carefully that situation will not be occurred. because <div>
My suggestion isn't disturb the current behavior. </div><div><div><div><br></div><div>I just add a configuration file, and add  a check logic, </div><div>If there is no configuration file, nova-api execute current behavior normally.</div>
</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-02-04 Russell Bryant <span dir="ltr"><<a href="mailto:rbryant@redhat.com" target="_blank">rbryant@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 02/03/2014 07:25 PM, Jae Sang Lee wrote:<br>
> Hi, Stackers.<br>
><br>
> The deadline for icehouse comes really quickly and I understand that there<br>
> are a lot of work todo, but I would like get your attention about my<br>
> blueprint for configurable locked vm api.<br>
><br>
>  - <a href="https://blueprints.launchpad.net/nova/+spec/configurable-locked-vm-api" target="_blank">https://blueprints.launchpad.net/nova/+spec/configurable-locked-vm-api</a><br>
><br>
> So far, developer places the decoration(@check_instance_lock) in the<br>
> function's declaration,<br>
> for example)<br>
>     @wrap_check_policy<br>
>     @check_instance_lock<br>
>     @check_instance_cell<br>
>     @check_instance_state(vm_state=None, task_state=None,<br>
>                           must_have_launched=False)<br>
>     def delete(self, context, instance):<br>
>         """Terminate an instance."""<br>
>         LOG.debug(_("Going to try to terminate instance"),<br>
> instance=instance)<br>
>         self._delete_instance(context, instance)<br>
><br>
> So good, but when administrator want to change API policy for locked vm,<br>
> admin must modify source code, and restart.<br>
><br>
> I suggest nova api do check api list for locked vm using config file<br>
> like policy.json. It just modify a config file, not a code<br>
> and don't need to service restart.<br>
><br>
> Can you take a small amount of time to approve a blueprint for icehouse-3?<br>
<br>
</div></div>I'm concerned about this idea from an interop perspective.  It means<br>
that "lock" will not mean the same thing from one cloud to another.<br>
That seems like something we should avoid.<br>
<br>
One thing that might work is to do this from the API side.  We could<br>
allow the caller of the API to list which operations are locked.  The<br>
default behavior would be the current behavior of locking all<br>
operations.  That gives some flexibility and keeps the API call working<br>
the same way across clouds.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Russell Bryant<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<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>
</font></span></blockquote></div><br></div>