iwasakims commented on pull request #838:
URL: https://github.com/apache/bigtop/pull/838#issuecomment-986481472


   
   @elukey
   The fix looks intended for running puppetize.sh again to refresh existing 
environment. If it is the first time, the latest ca-certificates is installed 
regardless of your fix since it is required by (rubygems required by) puppet.
   
   ```
   $ docker run bigtop/puppet:trunk-centos-7 rpm -q --whatrequires 
ca-certificates
   openssl-libs-1.0.2k-19.el7.x86_64
   rubygems-2.0.14.1-36.el7.noarch
   ```
   
   I think `./gradlew toolchain` tends to be used to refresh already existing 
dev env rather than puppetize.sh. Should we add `yum update ca-certificates` in 
the manifests under 
[bigtop_toolchain/manifests](https://github.com/apache/bigtop/tree/master/bigtop_toolchain/manifests)
 too?
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to