Ironic httpboot path change breaks non-containerized undercloud

[Ben Nemec]

https://github.com/openstack/python-tripleoclient/commit/1e2af1aeb31be7168780d2bc053210a94f5fcc55 is a breaking change for the non-containerized undercloud (which is why it got reverted in the first place, but I assume ci is now either all containers or has a workaround). If a different default is needed for containers then logic should be added to address that IMHO.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/584154

[Bogdan Dobrelya (bogdando)]

Note, instack undercloud deployments will not be supported for Rocky. For this milestone-3, we're doing the final steps for switching all CI jobs (but one) to containerized UC.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/584245

[OpenStack Infra (hudson-openstack)]

Change abandoned by Bogdan Dobrelya (<email address hidden>) on branch: master
Review: https://review.openstack.org/584245
Reason: it is already automated in upgrade tasks http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/docker/services/ironic-conductor.yaml#n204

[Bogdan Dobrelya (bogdando)]

Note, the upgrade impact of the change is covered in upgrade tasks http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/docker/services/ironic-conductor.yaml#n204

[Ben Nemec (bnemec)]

That's not what the release notes say: "instack-undercloud is deprecated in Rocky cycle and is replaced by the containerized undercloud efforts in python-tripleoclient."

Deprecated does not mean unsupported. If it is truly unsupported then we should just remove the option to use it at all. Either way this needs to be addressed.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/584154
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=54ec954a1eb490b4577cb6b3f971b4bac4c2355f
Submitter: Zuul
Branch: master

commit 54ec954a1eb490b4577cb6b3f971b4bac4c2355f
Author: Ben Nemec <email address hidden>
Date: Thu Jul 19 22:37:52 2018 +0000

    Move ironic http boot reno to the correct section

    This is not a deprecation. A deprecation leaves existing functionality
    in place but warns that it will go away in the future. This is a
    breaking change with upgrade impacts for anyone using a
    non-containerized undercloud.

    Change-Id: I66ae303d4d52bd2379f4aa8fe082705bae362049
    Related-Bug: 1782663

[Marios Andreou (marios-b)]

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

[Launchpad Janitor (janitor)]

[Expired for tripleo because there has been no activity for 60 days.]