Don't modify the docker unit file

Bug #1809361 reported by Tim Van Steenburgh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canal Charm
New
Undecided
Unassigned

Bug Description

Opened by g-ramirez on 2018-09-25 17:23:05+00:00 at https://github.com/juju-solutions/layer-canal/issues/20

------------------------------------------------------------

Hello,
It appears the canal charm overwrites the proxy settings from the principal units on the docker daemon.
Steps to reproduce:
juju deploy etcd
juju deploy kubernetes-master
juju deploy kubernetes-worker
juju config kubernetes-worker no_proxy= http_proxy= https_proxy=
juju deploy canal
juju add-relation canal etcd
juju add-relation canal kubernetes-master
juju add-relation canal kubernetes-worker
The below logic is already handled by the principal charm, so having it on the subordinate will overwrite its settings:
@when_any('config.changed.http_proxy', 'config.changed.https_proxy',
'config.changed.no_proxy')
@when('docker.ready')
def proxy_changed():
'''The proxy information has changed, render templates and restart the
docker daemon.'''
recycle_daemon()
https://api.jujucharms.com/charmstore/v5/~containers/canal/archive/reactive/docker.py
https://api.jujucharms.com/charmstore/v5/~containers/kubernetes-worker/archive/reactive/docker.py
Thanks,
Gabe

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.