ntp is contraindicated in 18.04 and later

Bug #1915351 reported by Vern Hart
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceph RADOS Gateway Charm
Fix Released
Wishlist
Chris MacNaughton

Bug Description

The ceph-radosgw charm installs the ntp package (unless it's running in a container).

The ntp provides ntpd which I understand doesn't get security updates in bionic or newer releases.

Additionally, in most deployments we have the ntp charm which recommends chrony. The ntp charm likely will install chrony after the ceph-radosgw charm installs ntp. The latter will get removed. However, as soon as something triggers the config-changed hook, the ceph-radosgw charm will install ntp and remove chrony. We'll go back the other way if/when the ntp charm is upgraded.

Can we make the charm intelligent about not installing ntp if chrony is already installed?

It might also be a good idea to make the charm pick the package conditionally, like how the ntp charm does it, chosing chrony if centos or ubuntu bionic or later. Everything else gets ntp.

Or maybe it would be better to remove the ntp package requirement altogether and instead require the ntp subordinate charm? I'm not sure how one would do this.

Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :

The other Ceph charms already have similar handling, such as not installing NTP in the charm (https://github.com/openstack/charm-ceph-mon/commit/37fbda6bd89090a15f03ae38372f93373a240cb0) as well as actually removing Chrony if it's inside of a container (https://github.com/openstack/charm-ceph-mon/commit/60a9a4f27aa1cddd717431abae7a26c46f5f8bd5).

Given the reasoning on the linked commits, adding similar changes to ceph-radosgw is a good idea.

Changed in charm-ceph-radosgw:
status: New → Confirmed
importance: Undecided → Wishlist
Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :
Changed in charm-ceph-radosgw:
assignee: nobody → Chris MacNaughton (chris.macnaughton)
milestone: none → 21.04
Changed in charm-ceph-radosgw:
status: Confirmed → Fix Committed
Changed in charm-ceph-radosgw:
status: Fix Committed → Fix Released
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.