[GitHub] [bigtop] elukey commented on pull request #754: (BIGTOP-3525) Replace hiera() with lookup()

[GitBox <gi...@apache.org>]

elukey commented on pull request #754:
URL: https://github.com/apache/bigtop/pull/754#issuecomment-797321967


   With a more fresh mind, Debian 10 installs puppet 5.5, so lookup in there is probably different and not comparable. It seems as if the lookup in puppet 3.6.2 doesn't really accept the default_value, ending up in not finding anything. 
   
   Debian 9 has also different problems (with puppet 4), see:
   
   ```
   Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, Lookup of key 'hadoop::kerberos_realm' failed: DataBinding 'hiera': Invalid interpolation method 'lookup' at /bigtop-home/bigtop-deploy/puppet/modules/hadoop/manifests/init.pp:35:7  at /bigtop-home/bigtop-deploy/puppet/manifests/cluster.pp:230 on node b27e057e3765.bigtop.apache.org
   ```
   
   Will  try to see if anything can be done, but maybe it is a task too difficult for the broad range of OSes supported, not sure :(


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org