Bug 1002928 - ntp is removed by default
Summary: ntp is removed by default
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: scheduler
Version: 0.13
Hardware: Unspecified
OS: Unspecified
urgent
high
Target Milestone: 0.14.2
Assignee: Amit Saha
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-30 09:16 UTC by Petr Sklenar
Modified: 2018-02-06 00:41 UTC (History)
14 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-11-07 01:46:47 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 901670 0 unspecified CLOSED Need a network time daemon to set clocks 2021-02-22 00:41:40 UTC

Internal Links: 901670

Description Petr Sklenar 2013-08-30 09:16:35 UTC
Description of problem:
ntp is removed by default

Version-Release number of selected component (if applicable):
Version - 0.14.1

How reproducible:
always

Steps to Reproduce:
1. https://beaker.engineering.redhat.com/jobs/466543
see kickstart
2. there is '-ntp' in the kickstart

Actual results:
'-ntp' in the kickstart

Expected results:
do not remove ntp

Additional info:
there are more similar jobs:
F19 https://beaker.engineering.redhat.com/jobs/466543
rhel7: https://beaker.engineering.redhat.com/jobs/483862

Comment 2 Bill Peck 2013-08-30 12:53:59 UTC
Looking at the Fedora kickstart I see this:

{% if no_clock_sync is undefined %}
chrony
-ntp
{% endif %}


I'm guessing you can pass no_clock_sync=1 in ks_meta and this won't happen.

I'm not sure why ntp is removed but I do know that chrony is the default in fedora going forward.

I'll let the beaker devs comment on if we really need the -ntp in there.

Comment 3 Nick Coghlan 2013-09-02 01:43:46 UTC
The removal of ntp when enabling chrony was part of Bill's original patch (http://gerrit.beaker-project.org/#/c/1889) for bug 901670 (which ensures systems have a time daemon by default, and the chosen daemon is chrony since that's the direction Fedora is heading).

This change was released in Beaker 0.13: http://beaker-project.org/docs/whats-new/release-0.13.html#chrony-is-enabled-when-appropriate

Having chrony and ntp fighting for control of the system clock seemed unwise, so the explicit removal of ntp was never questioned during the patch review.

If our understanding is incorrect, and it's possible for chrony and ntp to coexist happily on a single system, then we could change the default behaviour.

Alternatively, if it's only a specific subset of tests that need to avoid having chrony configured, then setting 'ks_meta="no_clock_sync"' to turn off the default snippet is designed to handle that use case.

Comment 4 Bill Peck 2013-09-02 02:51:37 UTC
Sorry Nick, I couldn't remember if I had included that or not.  I should have checked git's history.

Comment 5 Nick Coghlan 2013-09-03 01:09:48 UTC
Petr, does the no_clock_sync meta variable cover your use case?

It's specifically intended to cover tasks that want to configure the NTP related packages themselves, and really don't want Beaker messing with the time related system settings.

By default, though, we need to try to ensure the clock is properly synchronised (regardless of the underlying hardware), since it avoids many other problems with multi-host testing and aligning log timestamps.

Comment 6 Petr Sklenar 2013-09-03 08:35:39 UTC
Actually we are trying to test rhel in default installation. I guess its without negative package in the kickstart. 

The biggest issue is that if we have test which need package "ntp" (written in requires) its not installed.

So is that a way to remove ntp only when there is no test which requires 'ntp' without usage of 'no_clock_sync' ? 

+ developer of ntp and chrony says that there could be both packages installed but only one can be running. So what about to switch ntp off after the installation instead of removal.

Comment 7 Nick Coghlan 2013-09-03 08:50:32 UTC
OK, sounds good - we can switch the default configuration to ensuring ntp is disabled rather than removing it entirely.

no_clock_sync will then only be needed for any tests that actually need a default NTP configuration that Beaker hasn't tinkered with, rather than all tests that happen to have a nominal dependency on the ntp package (but can tolerate having chrony running instead).

Comment 8 Amit Saha 2013-09-10 06:42:30 UTC
Discovered something else to work on. So un-assigning myself for now.

Comment 9 Amit Saha 2013-09-12 03:39:37 UTC
On Gerrit: http://gerrit.beaker-project.org/#/c/2241/

Comment 13 Nick Coghlan 2013-10-03 02:27:50 UTC
Beaker 0.15 has been released.

Comment 14 Raymond Mancy 2013-10-23 01:56:32 UTC
This change has been nominated to be back ported to the 0.14 branch, to be released as part of the next maintenance release 0.14.2.

Comment 15 Nick Coghlan 2013-10-25 06:35:52 UTC
Adjusting target milestone to make the changes backported to 0.14.2 easier to identify. 0.15.0 has enough significant regressions that it shouldn't be used, so the change means that 0.15.1 can be effectively reidentified as the union of that tag and the 0.14.2 target milestone.

Comment 18 Nick Coghlan 2013-11-07 01:46:47 UTC
Closing as addressed in Beaker 0.14.2.


Note You need to log in before you can comment on or make changes to this bug.