Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-2381

Making Zookeeper hosts templatize in puppet module of hue

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: hue
    • Labels:
      None

      Description

      Zookeeper section in modules/hue/templates/hue.ini doesn't have host_ports templatize. If we can make it templatize then we can easily externalize zookeeper host/port configuration.

        Activity

        Hide
        amitkabraiiit Amit Kabra added a comment -

        Andrew Purtell : I have added the patch here but can't assign this jira to myself. Can you please assign it to me and review the patch. Thank you.

        Show
        amitkabraiiit Amit Kabra added a comment - Andrew Purtell : I have added the patch here but can't assign this jira to myself. Can you please assign it to me and review the patch. Thank you.
        Hide
        oflebbe Olaf Flebbe added a comment -

        I added you to contributors list and assigned the ticket to you. Looking into you contribution now.

        Show
        oflebbe Olaf Flebbe added a comment - I added you to contributors list and assigned the ticket to you. Looking into you contribution now.
        Hide
        oflebbe Olaf Flebbe added a comment -

        LGTM, +1

        Committing it in a minute.

        Show
        oflebbe Olaf Flebbe added a comment - LGTM, +1 Committing it in a minute.
        Hide
        oflebbe Olaf Flebbe added a comment -

        Amit Kabra: It would be easier for us if you can format your patch with git format / or use github pull requests . So I had to fake a git commit out of your patch file.

        But anyway, thanks a lot for your patch!

        Show
        oflebbe Olaf Flebbe added a comment - Amit Kabra : It would be easier for us if you can format your patch with git format / or use github pull requests . So I had to fake a git commit out of your patch file. But anyway, thanks a lot for your patch!

          People

          • Assignee:
            amitkabraiiit Amit Kabra
            Reporter:
            amitkabraiiit Amit Kabra
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development