Comment 28 for bug 1525214

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/8.0)

Reviewed: https://review.openstack.org/272593
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=a317fe5453da56b54f35734ac27f51970a625a33
Submitter: Jenkins
Branch: stable/8.0

commit a317fe5453da56b54f35734ac27f51970a625a33
Author: Roman Podoliaka <email address hidden>
Date: Tue Jan 26 17:23:42 2016 +0200

    Fix time sync after taking a snapshot

    Currently time sync is only performed for the admin node, as an empty
    list of nodes (default argument value) is passed to sync_time()
    method.

    On master branch this works as expected, as sync_time() behaviour
    was changed in commit 03159d5cfb740ec1fb41420d3867a2477f31b2d1. The
    latter seems to be a rather big change to be backported to the stable
    branch, though. The alternative is to fix this in 8.0 only by passing
    the list of nodes to sync_time().

    No commit is done to master first as API of sync_time() changed
    completely since 8.0 and those two would be entirely different
    changes (and master defaults to all nodes of the cluster - so
    we don't need to pass a list of nodes explicitly at all).

    Closes-Bug: #1525214

    Change-Id: Ibf639a5605f0ce29c9c3d3105b0c9c5880c8a965