Comment 2 for bug 1746523

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

Reviewed: https://review.openstack.org/539583
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_nova/commit/?id=c41103ba702cd6fd85d8e4d4fe86caef81623f44
Submitter: Zuul
Branch: master

commit c41103ba702cd6fd85d8e4d4fe86caef81623f44
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.

    Closes-Bug: #1746523
    Change-Id: Ic39f884016a6ba0498110267027783001a087a1c