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

bigtop-utils won't detect JAVA_HOME in some cases

    Details

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

      Description

      When environment is spun-off using bigtop_toolchain for jdk7, bigtop-utils can not detect JAVA_HOME correctly.

      The JDK in this case is installed in /usr/lib/jdk1.7.0_60 and linked as /usr/lib/jdk7-latest. bigtop-utils doesn't check these locations.

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        I think the easiest fix would be to add /usr/lib/jdk7-latest to the list of candidate paths. Any objections?

        Show
        cos Konstantin Boudnik added a comment - I think the easiest fix would be to add /usr/lib/jdk7-latest to the list of candidate paths. Any objections?
        Hide
        mgrover Mark Grover added a comment -

        No objections. That sound perfect!

        Show
        mgrover Mark Grover added a comment - No objections. That sound perfect!
        Hide
        cos Konstantin Boudnik added a comment -

        I've checked the this fixes the issue for bigtop_toolchain provided JDK7

        Show
        cos Konstantin Boudnik added a comment - I've checked the this fixes the issue for bigtop_toolchain provided JDK7
        Hide
        cos Konstantin Boudnik added a comment - - edited

        Mark Grover if you find the attached patch suitable - could you please commit it for me? My LDAP creds are screwed up... again ;( Thanks in advance!

        Show
        cos Konstantin Boudnik added a comment - - edited Mark Grover if you find the attached patch suitable - could you please commit it for me? My LDAP creds are screwed up... again ;( Thanks in advance!
        Hide
        mgrover Mark Grover added a comment -

        Sure thing, I will take a look soon! Thanks Cos!

        Show
        mgrover Mark Grover added a comment - Sure thing, I will take a look soon! Thanks Cos!
        Hide
        mgrover Mark Grover added a comment -

        +1 and committed. Thank you, Cos!

        Show
        mgrover Mark Grover added a comment - +1 and committed. Thank you, Cos!
        Hide
        cos Konstantin Boudnik added a comment -

        Ah - thanks for committing the patch Mark!

        Show
        cos Konstantin Boudnik added a comment - Ah - thanks for committing the patch Mark!

          People

          • Assignee:
            cos Konstantin Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development