diff options
author | Joshua Harlow <harlowja@gmail.com> | 2014-10-18 09:27:47 -0700 |
---|---|---|
committer | Joshua Harlow <harlowja@gmail.com> | 2014-10-18 09:27:47 -0700 |
commit | 2d9d3811b4b1d4ea078a0bba6cf5e067339c14f3 (patch) | |
tree | 1ce9fa86047650a8fecbacb4aae18272e22b1ab5 /cloudinit/util.py | |
parent | 797de394e5395f39b7f17403999e25cbe7f7a126 (diff) | |
download | vyos-cloud-init-2d9d3811b4b1d4ea078a0bba6cf5e067339c14f3.tar.gz vyos-cloud-init-2d9d3811b4b1d4ea078a0bba6cf5e067339c14f3.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 f236d0bf..13084374 100644 --- a/cloudinit/util.py +++ b/cloudinit/util.py @@ -1270,14 +1270,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): |