On one of our meetings  we agreed on keeping *ED states (DELETED,
INSPECTED, CLEANED, etc) as no-ops for now. Since then, however, the
inspection patch  got several comments from cores requesting removal
of INSPECTED state. That was done by Nisha.
Today we decided to approve  and bring the discussion here. We can
always add the state later, and blocking this patch again would be a bit
unfair IMO. We'll create a follow-up to  if we decide that we need
So now, are we keeping/adding these *ED states to our state machine? I
personally agree with what was discussed on the meeting, namely:
1. Keep *ED states
2. Make them no-ops, so that code that does INSPECTING -> MANAGEABLE
right now will do INSPECTING -> INSPECTED -> MANAGEABLE instead.
Having INSPECTED is also useful for distinguish between OOB case (when
inspect_hardware returns after having everything done) and in-band case
(when inspect_hardware returns after initializing inspection). We could
use return value of inspect_hardware being INSPECTING or INSPECTED.
OpenStack Development Mailing List (not for usage questions)