Github user JunHe77 commented on a diff in the pull request:

    https://github.com/apache/bigtop/pull/402#discussion_r222539413
  
    --- 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 --
    
    rpm base distros support using $basearch to automatically load arch 
dependent repo:
    
https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/6/html/deployment_guide/sec-using_yum_variables
    while for apt it doesn't use this way.
    
    The target is to unified configurations/deployment across different 
architectures for a same distro. So this patch starts from rpm based distros 
(centos/fedora/opensuse)


---

Reply via email to