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