Make novaless undercloud the default

[Steve Baker]

It is time to switch off nova/glance on the undercloud by default and have a separate metalsmith provision step be the only way to deploy managed baremetal.

The work items to do this would be

- Switch nova off in undercloud.conf by default, make sure quickstart is explicitly switching it on if/where required
- Reorganize tripleo-docs so that the overcloud deploy command is always passed deployed-server config, and emphasise the provision command for generating that config.
- Add a check in the overcloud deploy command which fails if there is no nova endpoint and deployed-server config is missing

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/749678

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/749679

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/749692

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/749812

[Kevin Carter (kevin-carter)]

ignore review 749812, it was added here in a mistaken review.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/749678
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=168961d66d0cbf089224848c66bc453677fe1748
Submitter: Zuul
Branch: master

commit 168961d66d0cbf089224848c66bc453677fe1748
Author: Rabi Mishra <email address hidden>
Date: Thu Sep 3 14:02:31 2020 +0530

    Add environment to enable nova on undercloud

    This sets the nova/glance services to OS::Heat::None in
    default undercloud environment and adds environment file to
    enable nova (if needed).

    Once tripleoclient has been changed to flip the nova_enable
    flag, we can drop undercloud-disable-nova.yaml

    Partial-Bug : #1891242
    Depends-On: https://review.opendev.org/#/c/749659/
    Change-Id: I88aaa58f49eb8a2dc38232132d0397a83c76104e

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/749679
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=30bc8b15b0bd57b24d31e029a1aa424be2debc34
Submitter: Zuul
Branch: master

commit 30bc8b15b0bd57b24d31e029a1aa424be2debc34
Author: Rabi Mishra <email address hidden>
Date: Thu Sep 3 14:20:55 2020 +0530

    Disable nova by default on undercloud

    Depends-On: https://review.opendev.org/749678
    Partial-Bug: #1891242
    Change-Id: Ifa4a94a6958bc8f5d16d4016ba8d4a6e3ca6f647

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/750271

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/750566
Committed: https://git.openstack.org/cgit/openstack/tripleo-quickstart-extras/commit/?id=339c2fe46f41516908b5eb7b2d88807264b63f94
Submitter: Zuul
Branch: master

commit 339c2fe46f41516908b5eb7b2d88807264b63f94
Author: yatinkarel <email address hidden>
Date: Wed Sep 9 10:52:19 2020 +0530

    Run flavor prep tasks only when nova is enabled

    These tasks are not needed when nova is not setup on
    undercloud.

    Related-Bug: #1891242
    Closes-Bug: #1894941
    Change-Id: Ie8b95ade76b1d13c1049a019fe83c04128a1e58d

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/750271
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=5a1fc3c983ab702a7a0d0f13aaddd6d0b8e3acfb
Submitter: Zuul
Branch: master

commit 5a1fc3c983ab702a7a0d0f13aaddd6d0b8e3acfb
Author: Rabi Mishra <email address hidden>
Date: Tue Sep 8 09:06:07 2020 +0530

    Drop undercloud-disable-nova.yaml

    Nova is disabled on undercloud by default and there is
    a new env file to enable it.

    Depends-On: https://review.opendev.org/#/c/749876/
    Closes-Bug: #1891242
    Change-Id: Iaa5566131c5a5d00889f5cf1078f2b825f2b0325