ASF GitHub Bot
added a comment - 06/Jun/16 22:19 Github user c0s commented on the issue:
https://github.com/apache/bigtop/pull/114
Interesting, I have fixed this back in Jan as
a97479a9d37d7c94f5128f36df2bad8b3676d2c2
Are you saying this change is gone?

So I'm guessing puppet installs the zeppelin deb (which does the right thing), but then it goes off and overwrites the /etc/zeppelin/conf/zeppelin-env.sh with its template from ./bigtop-deploy/puppet/modules/zeppelin/templates.

ASF GitHub Bot
added a comment - 06/Jun/16 22:40 Github user kwmonroe commented on the issue:
https://github.com/apache/bigtop/pull/114
Negative c0s, your commit is in there, but that only appears to affect the zeppelin deb. Your change to zeppelin-env.sh is not present in the puppet:
https://github.com/apache/bigtop/blob/master/bigtop-deploy/puppet/modules/zeppelin/templates/zeppelin-env.sh
So I'm guessing puppet installs the zeppelin deb (which does the right thing), but then it goes off and overwrites the /etc/zeppelin/conf/zeppelin-env.sh with its template from ./bigtop-deploy/puppet/modules/zeppelin/templates.
This PR is to ensure your changes from:
bigtop-packages/src/common/zeppelin/zeppelin-env.sh
make it into:
bigtop-deploy/puppet/modules/zeppelin/templates/zeppelin-env.sh