diff --git a/cluster/juju/layers/kubernetes-worker/layer.yaml b/cluster/juju/layers/kubernetes-worker/layer.yaml index 014fec02bac..d2ae0e3d61d 100644 --- a/cluster/juju/layers/kubernetes-worker/layer.yaml +++ b/cluster/juju/layers/kubernetes-worker/layer.yaml @@ -12,6 +12,9 @@ includes: - 'interface:kubernetes-cni' - 'interface:kube-dns' - 'interface:kube-control' +config: + deletes: + - install_from_upstream options: basic: packages: diff --git a/cluster/juju/layers/kubernetes-worker/reactive/kubernetes_worker.py b/cluster/juju/layers/kubernetes-worker/reactive/kubernetes_worker.py index c3501c97cd3..d42c9241a6c 100644 --- a/cluster/juju/layers/kubernetes-worker/reactive/kubernetes_worker.py +++ b/cluster/juju/layers/kubernetes-worker/reactive/kubernetes_worker.py @@ -39,7 +39,6 @@ from charmhelpers.core import hookenv, unitdata from charmhelpers.core.host import service_stop, service_restart from charmhelpers.contrib.charmsupport import nrpe - # Override the default nagios shortname regex to allow periods, which we # need because our bin names contain them (e.g. 'snap.foo.daemon'). The # default regex in charmhelpers doesn't allow periods, but nagios itself does. @@ -54,6 +53,10 @@ db = unitdata.kv() @hook('upgrade-charm') def upgrade_charm(): + # Trigger removal of PPA docker installation if it was previously set. + set_state('config.changed.install_from_upstream') + hookenv.atexit(remove_state, 'config.changed.install_from_upstream') + cleanup_pre_snap_services() check_resources_for_upgrade_needed()