[GitHub] [bigtop] MacChen01 commented on a diff in pull request #1147: BIGTOP-3882: add support for ambari

[via GitHub]

MacChen01 commented on code in PR #1147:
URL: https://github.com/apache/bigtop/pull/1147#discussion_r1301209454


##########
bigtop-deploy/puppet/modules/ambari/manifests/init.pp:
##########
@@ -23,6 +23,39 @@
   }
 
   class server {
+    #install python2 and init postgresql for openEuler
+    if $operatingsystem == 'openEuler' {
+      exec { "download_python2.7":

Review Comment:
   Ambari smoke test successlly using bigtop/puppet, the log is:
   
![image](https://github.com/apache/bigtop/assets/11262916/b416b6a5-75ea-4729-9d19-1f3651cbd284)
   
   The detail log is:
   
https://gitee.com/macchen1/bigtop/blob/master/smoke-test-log/openeuler-puppet-smoke-log/ambari-openeuler.log
   
   @guyuqi 
   We use the bigtop/puppet to smoke test is ok, please review the last PRs for 
openEuler.
   
   **BIGTOP-3988: using bigtop/puppet for hadoop:**
   https://github.com/apache/bigtop/pull/1168
   **BIGTOP-3824: add support for ranger**
   https://github.com/apache/bigtop/pull/1143
   **BIGTOP-3967: add support for spark**
   https://github.com/apache/bigtop/pull/1148
   
   We can verfiy all components if you check and merge the PRs into 
openEuler-support branch, and we can fix the problem if it happen basing the 
branch.
   Thanks.



-- 
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