diff options
author | Joshua Harlow <harlowja@yahoo-inc.com> | 2014-11-21 18:10:16 -0800 |
---|---|---|
committer | Joshua Harlow <harlowja@yahoo-inc.com> | 2014-11-21 18:10:16 -0800 |
commit | ad492708076e20c20f7c8ee96b1b2250bcad7d8f (patch) | |
tree | fc8eefe5b13c31af9bc8bb42b4919e28c41886a2 /cloudinit/util.py | |
parent | 61882a004937d5e6d922a6cdc4d65aa5a6315ea8 (diff) | |
parent | 2d9d3811b4b1d4ea078a0bba6cf5e067339c14f3 (diff) | |
download | vyos-cloud-init-ad492708076e20c20f7c8ee96b1b2250bcad7d8f.tar.gz vyos-cloud-init-ad492708076e20c20f7c8ee96b1b2250bcad7d8f.zip |
Pretty up the debug module
Previously the usage of the yaml_dumps module was causing
any python unicode key and value to show up as:
'item': !!python/unicode "some string"
This was not very pretty...
Fix this by using safe_dumps (which is also a good thing to
use and allow_unicode=True). Also create a tiny helper function
in the cc_debug module that does not include the yaml start and
end footers (since this module has its own footers and headers).
Also includes a basic sanity test for this module.
Diffstat (limited to 'cloudinit/util.py')
-rw-r--r-- | cloudinit/util.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/cloudinit/util.py b/cloudinit/util.py index b71057fb..ee5e5c0a 100644 --- a/cloudinit/util.py +++ b/cloudinit/util.py @@ -1274,14 +1274,14 @@ def read_write_cmdline_url(target_fn): logexc(LOG, "Failed writing url content to %s", target_fn) -def yaml_dumps(obj): - formatted = yaml.dump(obj, - line_break="\n", - indent=4, - explicit_start=True, - explicit_end=True, - default_flow_style=False) - return formatted +def yaml_dumps(obj, explicit_start=True, explicit_end=True): + return yaml.safe_dump(obj, + line_break="\n", + indent=4, + explicit_start=explicit_start, + explicit_end=explicit_end, + default_flow_style=False, + allow_unicode=True) def ensure_dir(path, mode=None): |