Github user JunHe77 commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/402#discussion_r222602799
--- 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 --
@epikhinm that's a good point. I didn't realize $ARCH can be used in repo
url. Just thought I can only specify what kind of arch a certain repo supports,
lie "arch=arch1,arch2".
Thanks for the suggestion and I will test and update new patch if it is OK
for auto loading.
---
