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

puppet: Update hue.ini to match Hue 3.7.1 template

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.0.0
    • Component/s: deployment, hue
    • Labels:
      None

      Description

      The hue.ini template in the hue puppet module is quite outdated with many options no longer used by Hue 3.7.1 and many missing sections that were added in later Hue releases.

      The hue.ini template should be remodeled to resemble the default Hue 3.7.1 template .

        Activity

        Hide
        petersla Peter Slawski added a comment -

        Attached patch that updates hue.ini template. It also comments out ldap_url as it shouldn't be set to a dummy url.

        Here is the diff between Bigtop's hue.ini (updated by this path) and the default hue.ini from Hue 3.7.1:
        https://www.diffchecker.com/2x9wq4gn

        Testing done

        For sanity, verified that I can continue to run Hive and Pig samples successfully, access HDFS using File Browser, and view jobs in Job Browser.

        Show
        petersla Peter Slawski added a comment - Attached patch that updates hue.ini template. It also comments out ldap_url as it shouldn't be set to a dummy url. Here is the diff between Bigtop's hue.ini (updated by this path) and the default hue.ini from Hue 3.7.1: https://www.diffchecker.com/2x9wq4gn Testing done For sanity, verified that I can continue to run Hive and Pig samples successfully, access HDFS using File Browser, and view jobs in Job Browser.
        Hide
        warwithin YoungWoo Kim added a comment - - edited

        Peter Slawski Thanks for the patch! Oveall LGTM, a few things:

        • Un-supported HUE apps by Bigtop. e.g., Impala, Security. should be added to app_blacklist? What do you think?

        and It would be nice if the template includes configurations related zookeeper (with another JIRA)

        Show
        warwithin YoungWoo Kim added a comment - - edited Peter Slawski Thanks for the patch! Oveall LGTM, a few things: Un-supported HUE apps by Bigtop. e.g., Impala, Security. should be added to app_blacklist? What do you think? and It would be nice if the template includes configurations related zookeeper (with another JIRA)
        Hide
        petersla Peter Slawski added a comment -

        Thanks YoungWoo Kim for taking a look!

        I think its a great idea overall to have the ability to customize which Hue apps appear in Hue via Puppet. One approach could be to blacklist undesired apps as you suggested via app_blacklist. The blacklist should be configurable with of course the default set to the unsupported Hue apps.

        Another approach would be to have a whitelist. I was aiming to do so in BIGTOP-1705. Here, you only install the Hue app packages you desire. The whitelist default could be set to the apps supported in Bigtop. However, I do see the blacklist approach being much simpler. It seems both effectively achieve the same thing internally from looking at the loading of apps in Hue:

        https://github.com/cloudera/hue/blob/release-3.7.1/desktop/core/src/desktop/appmanager.py#L230

        Yes, it would be nice to include configuration related to zookeeper in the template. Both seem appropriate to achieve in new JIRAs. I'll create them!

        Show
        petersla Peter Slawski added a comment - Thanks YoungWoo Kim for taking a look! I think its a great idea overall to have the ability to customize which Hue apps appear in Hue via Puppet. One approach could be to blacklist undesired apps as you suggested via app_blacklist. The blacklist should be configurable with of course the default set to the unsupported Hue apps. Another approach would be to have a whitelist. I was aiming to do so in BIGTOP-1705 . Here, you only install the Hue app packages you desire. The whitelist default could be set to the apps supported in Bigtop. However, I do see the blacklist approach being much simpler. It seems both effectively achieve the same thing internally from looking at the loading of apps in Hue: https://github.com/cloudera/hue/blob/release-3.7.1/desktop/core/src/desktop/appmanager.py#L230 Yes, it would be nice to include configuration related to zookeeper in the template. Both seem appropriate to achieve in new JIRAs. I'll create them!
        Hide
        warwithin YoungWoo Kim added a comment -

        Makes sense! Will commit this shortly. feel free to ping me if you need review and test for follow-ups as well.
        Thank you for your contribution!

        Show
        warwithin YoungWoo Kim added a comment - Makes sense! Will commit this shortly. feel free to ping me if you need review and test for follow-ups as well. Thank you for your contribution!
        Hide
        petersla Peter Slawski added a comment -

        Awesome! Thanks. I've created BIGTOP-1799 and BIGTOP-1800.

        Show
        petersla Peter Slawski added a comment - Awesome! Thanks. I've created BIGTOP-1799 and BIGTOP-1800 .

          People

          • Assignee:
            petersla Peter Slawski
            Reporter:
            petersla Peter Slawski
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development