Node update via "baremetal import" fails when some of the existing nodes are in not in "enroll" or "available" state

[Dmitry Tantsur]

Our workflows expect all nodes to be in these states. This is not correct, e.g. when some nodes are deployed to or are in manageable state. We need to only apply "manage" action to nodes in "enroll" state.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/392179

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/392653

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/392179
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=601a021b0a331fa31bc35984ad44d9eb87d9f031
Submitter: Jenkins
Branch: master

commit 601a021b0a331fa31bc35984ad44d9eb87d9f031
Author: Dmitry Tantsur <email address hidden>
Date: Tue Nov 1 15:29:09 2016 +0100

    Do not try "manage" actions on nodes that are not in "enroll" state

    When updating existing nodes, the register_or_update workflow tries
    "manage" action on them without considering their current state.
    This change fixes it by only running "manage" on nodes in "enroll" state.

    Partial-Bug: #1635240
    Change-Id: I38a52478a6c671c054d08dd15e9bf6226c5dc5d7

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/394429

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/394429
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=f1d85243094ab812122887d29485c8d3d46ab16b
Submitter: Jenkins
Branch: stable/newton

commit f1d85243094ab812122887d29485c8d3d46ab16b
Author: Dmitry Tantsur <email address hidden>
Date: Tue Nov 1 15:29:09 2016 +0100

    Do not try "manage" actions on nodes that are not in "enroll" state

    When updating existing nodes, the register_or_update workflow tries
    "manage" action on them without considering their current state.
    This change fixes it by only running "manage" on nodes in "enroll" state.

    Partial-Bug: #1635240
    Change-Id: I38a52478a6c671c054d08dd15e9bf6226c5dc5d7
    (cherry picked from commit 601a021b0a331fa31bc35984ad44d9eb87d9f031)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/392653
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=11e1c3c6100366a6a783c96b595310aecc38687c
Submitter: Jenkins
Branch: master

commit 11e1c3c6100366a6a783c96b595310aecc38687c
Author: Dmitry Tantsur <email address hidden>
Date: Wed Nov 2 14:18:18 2016 +0100

    Do not try to "provide" updated nodes not in "manageable" state

    The "import" command can update existing nodes, which are not necessary
    in "manageable" state. However, we try to unconditionally run the "provide"
    workflow for such nodes. This change limits it to only "manageable" nodes.

    Change-Id: I15219e53abeb43712b25cb3098334f4805f762b5
    Partial-Bug: #1635240

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/394930

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/394930
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=8a7f46b0be61d1a4882d88b1487c5f0065b78a64
Submitter: Jenkins
Branch: stable/newton

commit 8a7f46b0be61d1a4882d88b1487c5f0065b78a64
Author: Dmitry Tantsur <email address hidden>
Date: Wed Nov 2 14:18:18 2016 +0100

    Do not try to "provide" updated nodes not in "manageable" state

    The "import" command can update existing nodes, which are not necessary
    in "manageable" state. However, we try to unconditionally run the "provide"
    workflow for such nodes. This change limits it to only "manageable" nodes.

    Change-Id: I15219e53abeb43712b25cb3098334f4805f762b5
    Partial-Bug: #1635240
    (cherry picked from commit 11e1c3c6100366a6a783c96b595310aecc38687c)