Github user JunHe77 commented on a diff in the pull request: https://github.com/apache/bigtop/pull/402#discussion_r222637627 --- Diff: provisioner/docker/config_fedora-26.yaml --- @@ -14,10 +14,10 @@ # limitations under the License. docker: - memory_limit: "4g" - image: "bigtop/puppet:trunk-centos-6" + memory_limit: "8g" + image: "bigtop/puppet:1.3.0-fedora-26" -repo: "http://repos.bigtop.apache.org/releases/1.2.1/centos/6/x86_64" +repo: "http://repos.bigtop.apache.org/releases/1.3.0/fedora/26/$basearch" --- End diff -- Using $ARCH to automatically load different architecture repo on deb systems is verified on x86_64, aarch64 and ppc64le. Patch is updated. "docker-hadoop.sh" is modified to load correct puppet image by detecting CPU arch. This makes only one config file is needed for a certain distro acrossing different architectures. Pls review. Thanks.
---