cloud-init-dev team mailing list archive
-
cloud-init-dev team
-
Mailing list archive
-
Message #06821
Re: [Merge] ~smoser/cloud-init:fix/1849640-adjust-yaml-usage into cloud-init:master
OK. I just wanted to make sure that I wasn't missing something on the dump path.
For relocating; there are quite a few yaml users inside util.py so we'd still need to import
safeyaml into util; so module load wise, we're still pulling it in on any import of util. additionally, moving the load_yaml into safeyaml would mean inter-module deps; load_yaml uses decode_binary(), so we'd need to duplicate that.
I'm fine with this as it is, but would also review moving all yaml operations into safeyaml.
--
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/374679
Your team cloud-init Commiters is requested to review the proposed merge of ~smoser/cloud-init:fix/1849640-adjust-yaml-usage into cloud-init:master.
References