[ https://issues.apache.org/jira/browse/BIGTOP-1689?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14370968#comment-14370968 ]
Michael Weiser commented on BIGTOP-1689: ---------------------------------------- Sounds good to me. That program could also have a default not to allow overrides to avoid confusion but the option to allow them for very complex setups. That way it could easily be determined in bug reports if overrides are in play or not. I'd be happy to help in implementing the Puppet part of it. Also: Let's call the module and it's main class bigtop_util (underscore) because dashes are illegal in Puppet variable names and module class names become part of those when referenced with scopes (e.g. $bigtop_util::allow_overrides). > puppet: Allow merging arbitrary site configuration > -------------------------------------------------- > > Key: BIGTOP-1689 > URL: https://issues.apache.org/jira/browse/BIGTOP-1689 > Project: Bigtop > Issue Type: Improvement > Components: deployment > Affects Versions: 0.8.0 > Reporter: Peter Slawski > Assignee: Peter Slawski > Fix For: 1.0.0 > > Attachments: BIGTOP-1689.1.patch > > > Puppet should be flexible in allowing arbitrary configuration name value > pairs to be merged into a given site.xml file that was generated from a > template. > For example, the following could be included in site.yaml which would add a > configuration entry for hadoop.tmp.dir in core-site.xml: > {code} > hadoop::common_hdfs::hadoop_core_site_overrides: > "hadoop.tmp.dir": "/mnt/var/lib/hadoop/tmp" > {code} > This could be implemented as a puppet custom-function taking in the output of > template: > {code} > file { > "/etc/hadoop/conf/core-site.xml": > content => merge_site(template('hadoop/core-site.xml'), > $hadoop_core_site_overrides) > require => [Package["hadoop"]], > } > {code} > Perhaps another approach would be to have site.xml templates be created from > a single map of name value pairs. The merge would happen before the file > content is generated from the template. -- This message was sent by Atlassian JIRA (v6.3.4#6332)