diff options
author | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-05-20 11:02:56 +0200 |
---|---|---|
committer | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-05-20 11:02:56 +0200 |
commit | fb72d7dca65eefe06bec6880d12271c66b161efb (patch) | |
tree | 80e6faa362ac94a03f08f1b94664c02cd85c39f9 | |
parent | 9a8b501e5c66a1f31adcfcc342f677a6288188fc (diff) | |
download | ironic-fb72d7dca65eefe06bec6880d12271c66b161efb.tar.gz |
Trivial: comment why we don't check retired in allocations
Change-Id: I31e128f5273cc50bf7662a62080251d8f226f6c5
-rw-r--r-- | ironic/conductor/allocations.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/ironic/conductor/allocations.py b/ironic/conductor/allocations.py index 6cbe72be3..b6ce3ed1c 100644 --- a/ironic/conductor/allocations.py +++ b/ironic/conductor/allocations.py @@ -103,6 +103,8 @@ def _traits_match(traits, node): def _candidate_nodes(context, allocation): """Get a list of candidate nodes for the allocation.""" + # NOTE(dtantsur): not checking the retired flag because it's impossible + # (by the API contract) to have a retired node in the available state. filters = {'resource_class': allocation.resource_class, 'provision_state': states.AVAILABLE, 'associated': False, |