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).
Reviewed: https:/ /review. openstack. org/272593 /git.openstack. org/cgit/ openstack/ fuel-qa/ commit/ ?id=a317fe5453d a56b54f35734ac2 7f51970a625a33
Committed: https:/
Submitter: Jenkins
Branch: stable/8.0
commit a317fe5453da56b 54f35734ac27f51 970a625a33
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 1fb41420d3867a2 477f31b2d1. The
was changed in commit 03159d5cfb740ec
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: Ibf639a5605f0ce 29c9c3d3105b0c9 c5880c8a965