Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: vm
    • Labels:
      None

      Description

      I ran into following issue:

      $ vagrant up
      
      ...
      
      ==> bigtop1: Error: Unknown function is_array at /tmp/vagrant-puppet/manifests-f76206f1841b10444f0cea13160540f3/cluster.pp:24 on node bigtop1.vagrant
      ==> bigtop1: Error: Unknown function is_array at /tmp/vagrant-puppet/manifests-f76206f1841b10444f0cea13160540f3/cluster.pp:24 on node bigtop1.vagrant
      
      
      1. BIGTOP-1879.1.patch
        1 kB
        YoungWoo Kim
      2. BIGTOP-1879.2.patch
        2 kB
        YoungWoo Kim
      3. BIGTOP-1879.3.patch
        2 kB
        YoungWoo Kim

        Activity

        Hide
        warwithin YoungWoo Kim added a comment -

        BIGTOP-1879.1.patch

        • Add a installation step to pull puppet modules
        Show
        warwithin YoungWoo Kim added a comment - BIGTOP-1879.1.patch Add a installation step to pull puppet modules
        Hide
        warwithin YoungWoo Kim added a comment - - edited

        BIGTOP-1879.2.patch

        • Add a installation step to pull puppet modules
        • Create required dirs – /data/{1,2}
        Show
        warwithin YoungWoo Kim added a comment - - edited BIGTOP-1879.2.patch Add a installation step to pull puppet modules Create required dirs – /data/{1,2}
        Hide
        evans_ye Evans Ye added a comment - - edited

        Hey! Thanks for spotting this. I'm sorry that I broke this function in BIGTOP-1870...
        To make it right, I think we should add back puppet apply and mkdir -p /data/

        {1,2} in bigtop/bigtop-deploy/utils/setup-env-*.sh
        In addition, the patch shoud delete puppet apply and mkdir -p /data/{1,2}

        in bigtop/bigtop-deploy/vagrant-puppet-docker/provision.sh.
        This can make the feature consistent across docker and vagrant provisioner. How do you think YoungWoo Kim?

        Show
        evans_ye Evans Ye added a comment - - edited Hey! Thanks for spotting this. I'm sorry that I broke this function in BIGTOP-1870 ... To make it right, I think we should add back puppet apply and mkdir -p /data/ {1,2} in bigtop/bigtop-deploy/utils/setup-env-*.sh In addition, the patch shoud delete puppet apply and mkdir -p /data/{1,2} in bigtop/bigtop-deploy/vagrant-puppet-docker/provision.sh . This can make the feature consistent across docker and vagrant provisioner. How do you think YoungWoo Kim ?
        Hide
        warwithin YoungWoo Kim added a comment -

        Yup! Makes sense. I forgot to check the docker based VM. I'll revise the patch and upload updated patch shortly. Thanks.

        Show
        warwithin YoungWoo Kim added a comment - Yup! Makes sense. I forgot to check the docker based VM. I'll revise the patch and upload updated patch shortly. Thanks.
        Hide
        warwithin YoungWoo Kim added a comment -

        Evans Ye, Updated patch here BIGTOP-1879.3.patch. Thanks!

        Show
        warwithin YoungWoo Kim added a comment - Evans Ye , Updated patch here BIGTOP-1879.3.patch . Thanks!
        Hide
        evans_ye Evans Ye added a comment -

        The patch works like a charm!
        +1 and committed.
        Thanks for saving the feature YoungWoo Kim.

        Show
        evans_ye Evans Ye added a comment - The patch works like a charm! +1 and committed. Thanks for saving the feature YoungWoo Kim .

          People

          • Assignee:
            warwithin YoungWoo Kim
            Reporter:
            warwithin YoungWoo Kim
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development