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.
I think it still doesn't work even after https:/ /review. opendev. org/c/openstack /charms. ceph/+/ 792720. The logic seems wrong. Specifically:
if not install_list:
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.