Moved logs have potential to confuse users

[Jiří Stránský]

We should help users get oriented in the new log file locations when deploying containerized overcloud.

Original issue report here:

https://bugzilla.redhat.com/show_bug.cgi?id=1502860

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/518583
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=669be6f43652e6e7b244f5618de6145006df8b6c
Submitter: Zuul
Branch: master

commit 669be6f43652e6e7b244f5618de6145006df8b6c
Author: Jiri Stransky <email address hidden>
Date: Wed Nov 8 18:18:36 2017 +0100

    Add missing host_prep_tasks for log dirs creation

    Change-Id: Iecce2820657c5ad6bd2b02849c0433e5dce8009b
    Partial-Bug: #1730957

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/518729
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=db60db93c626bae029561ba5d3cd838668f6ddc2
Submitter: Zuul
Branch: master

commit db60db93c626bae029561ba5d3cd838668f6ddc2
Author: Jiri Stransky <email address hidden>
Date: Thu Nov 9 13:59:23 2017 +0100

    Add missing host_prep_tasks for memcached

    I missed this in change
    Iecce2820657c5ad6bd2b02849c0433e5dce8009b. Memcached seems to be the
    only service which mounts the /var/log/container/<service-name>
    directory directly under /var/log in the container.

    Change-Id: I26928bc7ad5f2f1bed60d200e8f2f0c310a14ee5
    Partial-Bug: #1730957

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/518584
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=85ec1934034a451ae810feb4c83ef5f351e6f8ce
Submitter: Zuul
Branch: master

commit 85ec1934034a451ae810feb4c83ef5f351e6f8ce
Author: Jiri Stransky <email address hidden>
Date: Wed Nov 8 18:19:25 2017 +0100

    Write readme.txt into old log directories

    This should help operators find the new log files. We do have them
    documented, but not everybody reads every word in the docs :)

    The readme creation has ignore_errors: true so that if the directory
    isn't present at all (e.g. on deployed server environments, which
    don't have openstack packages installed), we don't fail the deployment
    when we're not able to create the readme.

    Change-Id: I6b36db7b7ce8b3e4da566eb7828d0c3b8646a14f
    Partial-Bug: #1730957

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/520568

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/520569

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/520570

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/520568
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=de0141ccc4e5f6e85c249926440e6f0e4b718a3c
Submitter: Zuul
Branch: stable/pike

commit de0141ccc4e5f6e85c249926440e6f0e4b718a3c
Author: Jiri Stransky <email address hidden>
Date: Wed Nov 8 18:18:36 2017 +0100

    Add missing host_prep_tasks for log dirs creation

    Change-Id: Iecce2820657c5ad6bd2b02849c0433e5dce8009b
    Partial-Bug: #1730957
    (cherry picked from commit 669be6f43652e6e7b244f5618de6145006df8b6c)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/520569
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=30ab58fb989f5d79154cccf97fd8780ce952839c
Submitter: Zuul
Branch: stable/pike

commit 30ab58fb989f5d79154cccf97fd8780ce952839c
Author: Jiri Stransky <email address hidden>
Date: Thu Nov 9 13:59:23 2017 +0100

    Add missing host_prep_tasks for memcached

    I missed this in change
    Iecce2820657c5ad6bd2b02849c0433e5dce8009b. Memcached seems to be the
    only service which mounts the /var/log/container/<service-name>
    directory directly under /var/log in the container.

    Change-Id: I26928bc7ad5f2f1bed60d200e8f2f0c310a14ee5
    Partial-Bug: #1730957
    (cherry picked from commit db60db93c626bae029561ba5d3cd838668f6ddc2)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/520616
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=809c0d8bde3620564587ca62aae6316a6f89de08
Submitter: Zuul
Branch: master

commit 809c0d8bde3620564587ca62aae6316a6f89de08
Author: Jiri Stransky <email address hidden>
Date: Thu Nov 16 16:20:24 2017 +0100

    Add ignore_errors for memcached readme.txt

    This was supposed to be in I6b36db7b7ce8b3e4da566eb7828d0c3b8646a14f
    but i missed it on the edit because memcached doesn't have its own
    directory, so it didn't match the regex i created to edit the
    patch. Apologies for the patch churn on this. This fix has been
    directly incorporated into I6b36db7b7ce8b3e4da566eb7828d0c3b8646a14f
    backport, so we at least won't need to backport this oneliner.

    Change-Id: I31d8adf72ea6c3544e534dca57c5c415e2908674
    Partial-Bug: #1730957

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/520570
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=59d852836228103922001975748821d9e2a917d3
Submitter: Zuul
Branch: stable/pike

commit 59d852836228103922001975748821d9e2a917d3
Author: Jiri Stransky <email address hidden>
Date: Wed Nov 8 18:19:25 2017 +0100

    Write readme.txt into old log directories

    This should help operators find the new log files. We do have them
    documented, but not everybody reads every word in the docs :)

    The readme creation has ignore_errors: true so that if the directory
    isn't present at all (e.g. on deployed server environments, which
    don't have openstack packages installed), we don't fail the deployment
    when we're not able to create the readme.

    Change-Id: I6b36db7b7ce8b3e4da566eb7828d0c3b8646a14f
    Partial-Bug: #1730957
    (cherry picked from commit 85ec1934034a451ae810feb4c83ef5f351e6f8ce)

[OpenStack Infra (hudson-openstack)]

Change abandoned by Jiri Stransky (<email address hidden>) on branch: master
Review: https://review.openstack.org/519342
Reason: This is probably not worth it. We can un-abandon if needed.