overcloud node delete needs to support deployed-server delete by hostname

Would not it be better to change the documentation for deployed server scale down[1] above to create an environment with RemovalPolicies (with the to be blacklisted indexes) and use it rather than making an inefficient search for 'name' atrribute of the Deployedserver heat resource?

IMO, we should probably deprecate and remove the node delete command and ask users to use environments with RemovalPolicies.

The key issue with node delete is that, it internally calculates the role count and is different from the ones provided in env files or role_data and users have issues most of the time doing an update after scale down and failing if there are not enough baremetal nodes to provision.

I've submitted a WIP patch, if we still want to modify code to support host names. Not tested with deployed servers yet.

https://review.opendev.org/#/c/715129/

Reviewed: https://review.opendev.org/715129
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=be7c90e30c8a64969894fee8e424f665f1a5a00c
Submitter: Zuul
Branch: master

commit be7c90e30c8a64969894fee8e424f665f1a5a00c
Author: Rabi Mishra <email address hidden>
Date: Thu Mar 26 12:45:40 2020 +0530

    Match hostname when doing node delete

    ``overcloud node delete`` at present matches the provided nova
    instance ids with existing severs to build the heat blacklist.
    This patch would allow users to also use hostnames when deleting
    nodes. This will work with deployed servers and scale_playbook.yaml
    that matches host pattern.

    Partial-Bug: #1869089
    Change-Id: I3066689facc169125ade29156190cb98910f32b8

Fix proposed to branch: stable/train
Review: https://review.opendev.org/716468

Reviewed: https://review.opendev.org/716468
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=56c0fd5cafc75b1a5197897551c2224e14acdcad
Submitter: Zuul
Branch: stable/train

commit 56c0fd5cafc75b1a5197897551c2224e14acdcad
Author: Rabi Mishra <email address hidden>
Date: Thu Mar 26 12:45:40 2020 +0530

    Match hostname when doing node delete

    ``overcloud node delete`` at present matches the provided nova
    instance ids with existing severs to build the heat blacklist.
    This patch would allow users to also use hostnames when deleting
    nodes. This will work with deployed servers and scale_playbook.yaml
    that matches host pattern.

    Partial-Bug: #1869089
    Change-Id: I3066689facc169125ade29156190cb98910f32b8
    (cherry picked from commit be7c90e30c8a64969894fee8e424f665f1a5a00c)

Related fix proposed to branch: master
Review: https://review.opendev.org/717183

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/725608

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/725609

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/725620

Reviewed: https://review.opendev.org/725608
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=f6e3041291f2fdf082065aae1b80ee79a5870142
Submitter: Zuul
Branch: stable/stein

commit f6e3041291f2fdf082065aae1b80ee79a5870142
Author: Rabi Mishra <email address hidden>
Date: Thu Mar 26 12:45:40 2020 +0530

    Match hostname when doing node delete

    ``overcloud node delete`` at present matches the provided nova
    instance ids with existing severs to build the heat blacklist.
    This patch would allow users to also use hostnames when deleting
    nodes. This will work with deployed servers and scale_playbook.yaml
    that matches host pattern.

    Partial-Bug: #1869089
    Change-Id: I3066689facc169125ade29156190cb98910f32b8
    (cherry picked from commit be7c90e30c8a64969894fee8e424f665f1a5a00c)
    (cherry picked from commit 56c0fd5cafc75b1a5197897551c2224e14acdcad)

Reviewed: https://review.opendev.org/725609
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=ee6358f9e8aeb03b10c53bbb78eaf059f697cefd
Submitter: Zuul
Branch: stable/rocky

commit ee6358f9e8aeb03b10c53bbb78eaf059f697cefd
Author: Rabi Mishra <email address hidden>
Date: Thu Mar 26 12:45:40 2020 +0530

    Match hostname when doing node delete

    ``overcloud node delete`` at present matches the provided nova
    instance ids with existing severs to build the heat blacklist.
    This patch would allow users to also use hostnames when deleting
    nodes. This will work with deployed servers and scale_playbook.yaml
    that matches host pattern.

    Partial-Bug: #1869089
    Change-Id: I3066689facc169125ade29156190cb98910f32b8
    (cherry picked from commit be7c90e30c8a64969894fee8e424f665f1a5a00c)
    (cherry picked from commit 56c0fd5cafc75b1a5197897551c2224e14acdcad)

Reviewed: https://review.opendev.org/725620
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=927583ded34aa62c6c8220ed3e37a816454b40bb
Submitter: Zuul
Branch: stable/queens

commit 927583ded34aa62c6c8220ed3e37a816454b40bb
Author: Rabi Mishra <email address hidden>
Date: Thu Mar 26 12:45:40 2020 +0530

    Match hostname when doing node delete

    ``overcloud node delete`` at present matches the provided nova
    instance ids with existing severs to build the heat blacklist.
    This patch would allow users to also use hostnames when deleting
    nodes. This will work with deployed servers and scale_playbook.yaml
    that matches host pattern.

    Partial-Bug: #1869089
    Depends-On: https://review.opendev.org/727696
    Change-Id: I3066689facc169125ade29156190cb98910f32b8
    (cherry picked from commit be7c90e30c8a64969894fee8e424f665f1a5a00c)
    (cherry picked from commit 56c0fd5cafc75b1a5197897551c2224e14acdcad)

Change abandoned by Rabi Mishra (<email address hidden>) on branch: master
Review: https://review.opendev.org/717183

This is an automated action. Bug status has been set to 'Incomplete' and target milestone has been removed due to inactivity. If you disagree please re-set these values and reach out to us on freenode #tripleo