[openstack-dev] [Ironic] What to do with reservation check in node update API?
dtantsur at redhat.com
Tue Jul 21 13:41:59 UTC 2015
On 07/21/2015 03:26 PM, Lucas Alvares Gomes wrote:
>> So, it looks like the only reason we check the reservation field here is
>> because we want to return a 409 for "node is locked" rather than a 400,
>> right? do_node_deploy and such will raise a NodeLocked, which should do
>> the same as this check. It's unclear to me why we can't just remove this
>> check and let the conductor deal with it.
> Looking at the code this assumption seems to be correct. If the RPC
> methods raise NodeLocked we will return 409 as expected.
It's a bit more tricky, please see my patch:
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
More information about the OpenStack-dev