diff options
author | Joshua Harlow <harlowja@yahoo-inc.com> | 2012-08-10 10:41:52 -0700 |
---|---|---|
committer | Joshua Harlow <harlowja@yahoo-inc.com> | 2012-08-10 10:41:52 -0700 |
commit | e7116b69f093600e1e156ee332417d0a30b0df9c (patch) | |
tree | ab455104d38dcd37a707954924a8be7fba1a0777 /cloudinit/config | |
parent | d9a4aad5d5cc8cb78997d66850d1916341b8d1cd (diff) | |
download | vyos-cloud-init-e7116b69f093600e1e156ee332417d0a30b0df9c.tar.gz vyos-cloud-init-e7116b69f093600e1e156ee332417d0a30b0df9c.zip |
Adjustments so that the default old event is used when
no args are provided. Adjust the config file to use this
older method so that older changes aren't needed. Also
when calling 'initctl' emit fails log the exception message
so that some useful warning information can be produced.
Diffstat (limited to 'cloudinit/config')
-rw-r--r-- | cloudinit/config/cc_emit_upstart.py | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/cloudinit/config/cc_emit_upstart.py b/cloudinit/config/cc_emit_upstart.py index 79f7119f..68b86ff6 100644 --- a/cloudinit/config/cc_emit_upstart.py +++ b/cloudinit/config/cc_emit_upstart.py @@ -25,15 +25,15 @@ from cloudinit.settings import PER_ALWAYS frequency = PER_ALWAYS -distros = ['ubuntu'] +distros = ['ubuntu', 'debian'] def handle(name, _cfg, cloud, log, args): event_names = args if not event_names: - log.debug(("Skipping module named %s," - " no event names provided"), name) - return + # Default to the 'cloud-config' + # event for backwards compat. + event_names = ['cloud-config'] if not os.path.isfile("/sbin/initctl"): log.debug(("Skipping module named %s," " no /sbin/initctl located"), name) @@ -43,5 +43,6 @@ def handle(name, _cfg, cloud, log, args): cmd = ['initctl', 'emit', str(n), 'CLOUD_CFG=%s' % cfgpath] try: util.subp(cmd) - except: - log.warn("Emission of upstart event %s failed", n) + except Exception as e: + # TODO, use log exception from utils?? + log.warn("Emission of upstart event %s failed due to: %s", n, e) |