The proposal to merge ~chad.smith/cloud-init:cc-ntp-schema into cloud-init:master has been updated.
Description changed to: cc_ntp: Add schema definition and passive schema validation. cloud-config files are very flexible and permissive. This branch adds a supported schema definition to cc_ntp so that we can properly inform users of potential misconfiguration in their cloud-config yaml. This first branch adds a jsonsschema definition to the cc_ntp module and validation functions in cloudinit/config/schema which will log warnings about invalid configuration values in the ntp section. A cmdline tools/cloudconfig-schema is added which can be used in our dev environments to quickly attempt to exercise the ntp schema. Eventually this cloudconfig-schema tool will evolve into a cmdline tool we can provide to users or a validation service which will allow folks to test their cloud-config files without having to attempt deployment to find errors or inconsistencies. LP: #1692916 To test: # download the attached schemas.tar tar xf schemas.tar ./tools/cloudconfig-schema --doc # to dump rst from the schema definition ./tools/cloudconfig-schema --doc | rst2man | man -l - # manpage for file in `ls schemas`; do echo '--------------------------- BEGIN ' $file; cat schemas/$file; echo -e '--------------------------- OUTPUT: '; ./tools/cloudconfig-schema --config-file schemas/$file; echo -e '--------------------------- END ' $file '\n\n'; done; #run unit tests tox -- --tests tests/unittests/test_handlers/test_handler_ntp.py For more details, see: https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/324499 -- Your team cloud-init commiters is requested to review the proposed merge of ~chad.smith/cloud-init:cc-ntp-schema into cloud-init:master. _______________________________________________ Mailing list: https://launchpad.net/~cloud-init-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~cloud-init-dev More help : https://help.launchpad.net/ListHelp
