Comment 6 for bug 1746523

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_nova (stable/newton)

Reviewed: https://review.openstack.org/540331
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_nova/commit/?id=7cc81440be5d4834daef91130918a33fdc6eb3d7
Submitter: Zuul
Branch: stable/newton

commit 7cc81440be5d4834daef91130918a33fdc6eb3d7
Author: Jesse Pretorius <email address hidden>
Date: Wed Jan 31 15:49:44 2018 +0000

    Improve console install and restart services appropriately

    The current implementation wipes out the novnc/spice directory
    every single time the playbook executes. While it does replace
    it afterwards it does not fire a notifier to make the service
    restart, resulting in the service failing.

    This patch makes the install process idempotent - it will only
    wipe out the directory if it does not look the way it is
    expected to look. It also fires the notifier to restart services
    if the git repo changes.

    This should ensure that novnc/spice restart properly on upgrades
    and that there are no changes/restarts for the service if the git
    SHA or software does not change.

    Backport note:
    The 'Manage LB' handler notification is removed as it is not
    present in newton.

    Closes-Bug: #1746523
    Change-Id: Ic39f884016a6ba0498110267027783001a087a1c