time-sync in ntp.py must have support to sync time if server use chrony instead of ntpd

Bug #1583190 reported by Artem Hrechanychenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Anton Studenov
Mitaka
Confirmed
High
Dennis Dmitriev
Newton
Fix Committed
High
Anton Studenov

Bug Description

Detailed bug description:
In case if we use Centos Cloud image - chrony used as time-sync daemon instead of ntpd and if we revert such snapshot - we received traceback:
http://paste.openstack.org/show/497513/

reason- /etc/ntpd.conf is not present

after installing of fuel services - chrony is disabled and ntpd uses for time-syncing

Tags: area-qa
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-devops (master)

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

Changed in fuel:
assignee: Dennis Dmitriev (ddmitriev) → Anton Studenov (astudenov)
status: Confirmed → In Progress
Changed in fuel:
assignee: Anton Studenov (astudenov) → Alexey Stepanov (astepanov-m)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-devops (master)

Reviewed: https://review.openstack.org/321554
Committed: https://git.openstack.org/cgit/openstack/fuel-devops/commit/?id=67e4f9d8110014d7318b5ddbf64bc68587a3dbd9
Submitter: Jenkins
Branch: master

commit 67e4f9d8110014d7318b5ddbf64bc68587a3dbd9
Author: Anton Studenov <email address hidden>
Date: Fri May 27 13:06:30 2016 +0300

    Support for chrony NTP

    - added NtpChronyd
    - BaseNtp and its sub-classes were refactored
    - simplified AbstractNtp API
    - updated unit tests

    Closes-Bug: #1583190
    Change-Id: I10fcd0660a0fa20695b7e99f748e743f681bc21d

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-devops (release/2.9)

Fix proposed to branch: release/2.9
Review: https://review.openstack.org/342101

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-devops (release/2.9)

Reviewed: https://review.openstack.org/342101
Committed: https://git.openstack.org/cgit/openstack/fuel-devops/commit/?id=103706772165c31c984b77f74ef8fd9c1bb25616
Submitter: Jenkins
Branch: release/2.9

commit 103706772165c31c984b77f74ef8fd9c1bb25616
Author: Alexey Stepanov <email address hidden>
Date: Thu Jul 14 13:34:23 2016 +0300

    Support for chrony NTP

    - added NtpChronyd
    - BaseNtp and its sub-classes were refactored
    - simplified AbstractNtp API
    - updated unit tests

    Closes-Bug: #1583190
    Change-Id: I10fcd0660a0fa20695b7e99f748e743f681bc21d

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.