Comment 4 for bug 1929054

Revision history for this message
Liam Young (gnuoy) wrote :

I think it still doesn't work even after https://review.opendev.org/c/openstack/charms.ceph/+/792720. The logic seems wrong. Specifically:

         install_list = filter_missing_packages([CHRONY_PACKAGE])
         if not install_list:
             rm_packages.append(CHRONY_PACKAGE)

Since filter_missing_packages returns a list of packages which are installed this is only adding 'chrony' to the list if it is not present.