iwasakims commented on code in PR #1243: URL: https://github.com/apache/bigtop/pull/1243#discussion_r1513762357
########## bigtop_toolchain/bin/puppetize.sh: ########## @@ -69,7 +69,8 @@ case ${ID}-${VERSION_ID} in puppet module install puppetlabs-stdlib --version 4.12.0 # Enabling the PowerTools and EPEL repositories via Puppet doesn't seem to work in some cases. # As a workaround for that, enable the former here in advance of running the Puppet manifests. - dnf config-manager --set-enabled crb + dnf config-manager --set-enabled crb devel Review Comment: This should be done in the [Puppet manifest](https://github.com/apache/bigtop/blob/master/bigtop-deploy/puppet/modules/alluxio/manifests/init.pp)? We can not expect that the puppetize.sh is applied in all deployment. -- 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. To unsubscribe, e-mail: dev-unsubscr...@bigtop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org