← Back to team overview

cloud-init-dev team mailing list archive

Re: [Merge] lp:~daniel-thewatkins/cloud-init/merging-doc-clarification into lp:cloud-init


Diff comments:

> === modified file 'doc/merging.rst'
> --- doc/merging.rst	2013-04-02 18:51:54 +0000
> +++ doc/merging.rst	2016-05-26 11:24:31 +0000
> @@ -181,8 +182,13 @@
>  Other uses
>  ----------
> -The default merging algorithm for merging 'conf.d' yaml files (which form a initial
> -yaml config for cloud-init) was also changed to use this mechanism so its full
> +In addition to being used for merging user-data sections, the default merging
> +algorithm for merging 'conf.d' yaml files (which form an initial yaml config
> +for cloud-init) was also changed to use this mechanism so its full
>  benefits (and customization) can also be used there as well. Other places that
> -used the previous merging are also similar now extensible (metadata merging for
> -example).
> +used the previous merging are also, similarly, now extensible (metadata
> +merging, for example).
> +
> +Note, however, that merge algorithms are not used *across* types of
> +configuration.  As was previously the case, merged user-data will still

I find these words confusing "As was previously the case".  This makes it sound like a change was made, but I thought this was just meant to document existing behavior.

> +overwrite conf.d configuration.

Your team cloud init development team is requested to review the proposed merge of lp:~daniel-thewatkins/cloud-init/merging-doc-clarification into lp:cloud-init.