nova.conf override not used from documented location

[Raimund Hook]

As documented here:

https://docs.openstack.org/kolla-ansible/latest/admin/advanced-configuration.html#openstack-service-configuration-in-kolla

An override file for nova.conf should work if placed under the path: /etc/kolla/config/nova/nova.conf

During kolla-ansible execution, the any file placed here is in fact _not_ picked up.

[OpenStack Infra (hudson-openstack)]

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

[Raimund Hook (stingrayza)]

From IRC:

stingrayza happy to fix this, - the question is do I just do a quick doc clean-up, or do we fix the conf pick up to additionally pick up "{{ node_custom_config }}/nova/nova.conf" ? (and if the latter, then do I do that for _all_ services?) 10:54
mgoddard stingrayza: I think typically we don't support that path, although there may be exceptions. Worth checking a few to be sure, but I'd update docs 11:08
stingrayza mgoddard: cool. doc update inbound :)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/721241
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=08682243ed534366ff7ef52a538e14ad04d01f1c
Submitter: Zuul
Branch: master

commit 08682243ed534366ff7ef52a538e14ad04d01f1c
Author: Raimund Hook <email address hidden>
Date: Mon Apr 20 12:20:04 2020 +0100

    Update Advanced Config guide to clarify paths

    This update clears up an additional path that was mentioned in the
    Advanced Configuration documentation, but not actually picked up in the
    playbooks.

    This specifically affects Service Configuration overrides. The docs have
    been cleaned up to reflect the way the playbooks pick up the override
    files.

    Change-Id: Id15fe139af6462217c2ac26d7d21c5eac5368e12
    Closes-Bug: 1873782
    Signed-off-by: Raimund Hook <email address hidden>

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/726189
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=3efd5d6e190ae359b05ffa0e26cb28104cc6c8a2
Submitter: Zuul
Branch: stable/train

commit 3efd5d6e190ae359b05ffa0e26cb28104cc6c8a2
Author: Raimund Hook <email address hidden>
Date: Mon Apr 20 12:20:04 2020 +0100

    Update Advanced Config guide to clarify paths

    This update clears up an additional path that was mentioned in the
    Advanced Configuration documentation, but not actually picked up in the
    playbooks.

    This specifically affects Service Configuration overrides. The docs have
    been cleaned up to reflect the way the playbooks pick up the override
    files.

    Change-Id: Id15fe139af6462217c2ac26d7d21c5eac5368e12
    Closes-Bug: 1873782
    Signed-off-by: Raimund Hook <email address hidden>
    (cherry picked from commit 08682243ed534366ff7ef52a538e14ad04d01f1c)

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/729218
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=4b7dc5036bb4c83967a44ba15a5759400f6e27e1
Submitter: Zuul
Branch: stable/stein

commit 4b7dc5036bb4c83967a44ba15a5759400f6e27e1
Author: Raimund Hook <email address hidden>
Date: Mon Apr 20 12:20:04 2020 +0100

    Update Advanced Config guide to clarify paths

    This update clears up an additional path that was mentioned in the
    Advanced Configuration documentation, but not actually picked up in the
    playbooks.

    This specifically affects Service Configuration overrides. The docs have
    been cleaned up to reflect the way the playbooks pick up the override
    files.

    Change-Id: Id15fe139af6462217c2ac26d7d21c5eac5368e12
    Closes-Bug: 1873782
    Signed-off-by: Raimund Hook <email address hidden>
    (cherry picked from commit 08682243ed534366ff7ef52a538e14ad04d01f1c)