Bigtop
  1. Bigtop
  2. BIGTOP-794

puppet code around hadoop_head_node breaks on newer puppets

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 0.5.0
    • Fix Version/s: None
    • Component/s: Deployment
    • Labels:
      None

      Description

      At least on puppet 2.7.18 our puppet code breaks since in certain places it expects hadoop_head_node to be an array and in certain other places to be a string. We need to fix this before releasing 0.5.0 to the wild.

        Activity

        Roman Shaposhnik created issue -
        Hide
        Roman Shaposhnik added a comment -

        This gets us ublocked. Will keep posting here until all the kinks are ironed out.

        Show
        Roman Shaposhnik added a comment - This gets us ublocked. Will keep posting here until all the kinks are ironed out.
        Roman Shaposhnik made changes -
        Field Original Value New Value
        Attachment 0001-BIGTOP-794.-puppet-code-around-hadoop_head_node-brea.patch [ 12554909 ]
        Hide
        Patrick Taylor Ramsey added a comment -

        +1. Thanks for this, Roman.

        Show
        Patrick Taylor Ramsey added a comment - +1. Thanks for this, Roman.
        Hide
        Roman Shaposhnik added a comment -

        Here's one more piece to enable our Puppet on all the Linux distros. I've opted out to moving repo management outside of Puppet propper on non-yum platforms. It would be nice if somebody can help with implementing proper handling for zypper/apt-get as well.

        Show
        Roman Shaposhnik added a comment - Here's one more piece to enable our Puppet on all the Linux distros. I've opted out to moving repo management outside of Puppet propper on non-yum platforms. It would be nice if somebody can help with implementing proper handling for zypper/apt-get as well.
        Roman Shaposhnik made changes -
        Hide
        Patrick Taylor Ramsey added a comment -

        +1. LGTM.

        Show
        Patrick Taylor Ramsey added a comment - +1. LGTM.
        Hide
        Roman Shaposhnik added a comment -

        Attaching a crude Whirr-based script I've been using to test Bigtop on newer platforms. At some point it would be nice to have it folded into the project, I suppose.

        Show
        Roman Shaposhnik added a comment - Attaching a crude Whirr-based script I've been using to test Bigtop on newer platforms. At some point it would be nice to have it folded into the project, I suppose.
        Roman Shaposhnik made changes -
        Attachment launch.cluster.sh [ 12555095 ]
        Roman Shaposhnik made changes -
        Attachment launch.cluster.sh [ 12555095 ]
        Roman Shaposhnik made changes -
        Attachment launch.cluster.sh [ 12555096 ]
        Hide
        Mark Grover added a comment -

        Roman, I took a quick look at the script, looks good to me.
        One question: any particular reason why you are redirecting output of some of the commands to /etc/apt/apt.conf.d/9999force_yes instead of somewhere under /tmp?

        Show
        Mark Grover added a comment - Roman, I took a quick look at the script, looks good to me. One question: any particular reason why you are redirecting output of some of the commands to /etc/apt/apt.conf.d/9999force_yes instead of somewhere under /tmp ?
        Hide
        Roman Shaposhnik added a comment -

        Actually, this is not the redirection but rather how I create the content of the file /etc/apt/apt.conf.d/9999force_yes – the point of the exersize is to have that file in there so that apt gets to be friendlier and not asking so many questions.

        Show
        Roman Shaposhnik added a comment - Actually, this is not the redirection but rather how I create the content of the file /etc/apt/apt.conf.d/9999force_yes – the point of the exersize is to have that file in there so that apt gets to be friendlier and not asking so many questions.
        Hide
        Mark Grover added a comment -

        Ah, I see. Thanks!

        Show
        Mark Grover added a comment - Ah, I see. Thanks!
        Roman Shaposhnik made changes -
        Fix Version/s 0.6.0 [ 12323895 ]
        Fix Version/s 0.5.0 [ 12321865 ]
        Roman Shaposhnik made changes -
        Fix Version/s 0.6.0 [ 12323895 ]

          People

          • Assignee:
            Roman Shaposhnik
            Reporter:
            Roman Shaposhnik
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:

              Development