commit 1d27478e20239714a69cd219caba698989565ca8
Author: Ryan Bridges <email address hidden>
Date: Thu Feb 15 09:01:32 2018 -0800
Allow sqalchemy filtering by id and uuid
These additions will allow us to filter nodes
by node uuid and id. This filter API is used
in many places throughout the code base. It is
natural to expect that this API would allow us to
filter by node id and uuid in addtion to the other
supported parameters. This also fixes a 3 year old bug.
In conductor/manager.py, he calls _fail_if_in_state()
and uses the node_id as a filter. However this filter
effectively does nothing because sqalchemy does not
know how to filter by node id on the backend. My changes
fix this problem and make the API more intuitive
Closes-Bug: #1749755
Change-Id: I4efc0d5cd5d5d6108a334f954e1718203b47da0a
(cherry picked from commit 366a44a1bb4e235f6382af02962ba2a192cffa4d)
Reviewed: https:/ /review. openstack. org/545806 /git.openstack. org/cgit/ openstack/ ironic/ commit/ ?id=1d27478e202 39714a69cd219ca ba698989565ca8
Committed: https:/
Submitter: Zuul
Branch: stable/queens
commit 1d27478e2023971 4a69cd219caba69 8989565ca8
Author: Ryan Bridges <email address hidden>
Date: Thu Feb 15 09:01:32 2018 -0800
Allow sqalchemy filtering by id and uuid
These additions will allow us to filter nodes
by node uuid and id. This filter API is used
in many places throughout the code base. It is
natural to expect that this API would allow us to
filter by node id and uuid in addtion to the other
supported parameters. This also fixes a 3 year old bug.
This change from lucasagomes has a bug: /review. openstack. org/#/c/ 197141/
https:/
In conductor/ manager. py, he calls _fail_if_in_state()
and uses the node_id as a filter. However this filter
effectively does nothing because sqalchemy does not
know how to filter by node id on the backend. My changes
fix this problem and make the API more intuitive
Closes-Bug: #1749755
Change-Id: I4efc0d5cd5d5d6 108a334f954e171 8203b47da0a f6382af02962ba2 a192cffa4d)
(cherry picked from commit 366a44a1bb4e235