Comment 0 for bug 1739329

The configure_sources routine, in charmhelpers/fetch/__init__.py, exhibits surprising behavior that makes writing valid config.yaml and bundle.yaml files more difficult.

After the function retrieves a list of sources and keys from the config object, it calls yaml.safe_load on the values that it has retrieved. Te values were parsed as yaml when the config.yaml was initially read, so they effectively get parsed twice.

Most of the time, this does not cause problems, people use a one item or space separated list of simple strings when setting install_sources and install_keys, which survives the double parsing.

Things get weird when somebody does something more complicated, like embedding a GPG key in the config yaml. Since the yaml gets parsed twice, you need to populate install_keys with a string that contains a yaml formatted list. So the resultant yaml looks like this:

    install_keys: |
        - |
            -----BEGIN PGP PUBLIC KEY BLOCK-----
            ...

Note that the yaml list is escaped by a pipe. This is unintuitive and surprising, leading to confusion when creating config yaml files in the field.

The comments suggest that configure_sources behaves this way to work around a bug that the maintainers ran into in 2014. I suspect that there is a better way of addressing the bug, which leads to more intuitive, less surprising config.yamls and bundle.yamls. Thus this ticket :-)