Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.9.0
    • Fix Version/s: 1.0.0
    • Component/s: build
    • Labels:
      None

      Description

      bigtop-toolchain uses oracle jdk for all platforms. In BIGTOP-1480 it seems consensus to use openjdk7 instead.

      Since Oracle will stop free support of Oracle JDK 7 in near future it is better to rely on the distribution supplied openjdk 7 instead

        Issue Links

          Activity

          Hide
          cos Konstantin Boudnik added a comment -

          Agree. I think this is the absolute must to be able to use the same env. for CI and local development.

          Show
          cos Konstantin Boudnik added a comment - Agree. I think this is the absolute must to be able to use the same env. for CI and local development.
          Hide
          rvs Roman Shaposhnik added a comment -

          A big docker image is a huge nuisance for anybody trying to build locally on their laptop/desktop. I know Bigtop has this huge skew towards Jenkins workflow, but I think unless we can enable individual developers we're unlikely to get out of the current mindset of 'oh, its just too difficult to do anything locally'

          Show
          rvs Roman Shaposhnik added a comment - A big docker image is a huge nuisance for anybody trying to build locally on their laptop/desktop. I know Bigtop has this huge skew towards Jenkins workflow, but I think unless we can enable individual developers we're unlikely to get out of the current mindset of 'oh, its just too difficult to do anything locally'
          Hide
          jayunit100 jay vyas added a comment -

          i vote on the side of openjdk for sure !

          whats the big deal about a big docker image ? Its just a one time pull ? even if it takes 5 minutes,
          its not going to be something we have to manually sit there and watch .

          Show
          jayunit100 jay vyas added a comment - i vote on the side of openjdk for sure ! whats the big deal about a big docker image ? Its just a one time pull ? even if it takes 5 minutes, its not going to be something we have to manually sit there and watch .
          Hide
          cos Konstantin Boudnik added a comment -

          Argh.... on one hand we should have the same env. in CI and in the development. But looks like this is a real problem size-wise....

          Show
          cos Konstantin Boudnik added a comment - Argh.... on one hand we should have the same env. in CI and in the development. But looks like this is a real problem size-wise....
          Hide
          rvs Roman Shaposhnik added a comment -

          So here's a slight problem with this patch: it will make the size of our docker images WAY bigger, since the way OpenJDK is packages drags in so many useless dependencies I can't even start contemplating why all of them are needed.

          So... how do we deal with this?

          Show
          rvs Roman Shaposhnik added a comment - So here's a slight problem with this patch: it will make the size of our docker images WAY bigger, since the way OpenJDK is packages drags in so many useless dependencies I can't even start contemplating why all of them are needed. So... how do we deal with this?
          Hide
          cos Konstantin Boudnik added a comment -

          Please link them together.

          Show
          cos Konstantin Boudnik added a comment - Please link them together.
          Hide
          oflebbe Olaf Flebbe added a comment -

          Hm sorry guys. Forgot that I fixed JAVA_HOME, too.

          Have to open new ticket.

          Show
          oflebbe Olaf Flebbe added a comment - Hm sorry guys. Forgot that I fixed JAVA_HOME, too. Have to open new ticket.
          Hide
          cos Konstantin Boudnik added a comment -

          Pushed to the master. Thanks Olaf.

          Show
          cos Konstantin Boudnik added a comment - Pushed to the master. Thanks Olaf.
          Hide
          cos Konstantin Boudnik added a comment -

          Olaf Flebbe, the commit message should be the same as the JIRA title. I will fix this here manually though.

          Show
          cos Konstantin Boudnik added a comment - Olaf Flebbe , the commit message should be the same as the JIRA title. I will fix this here manually though.
          Hide
          cos Konstantin Boudnik added a comment -

          I haven't run it but the code looks correct. +1

          Show
          cos Konstantin Boudnik added a comment - I haven't run it but the code looks correct. +1
          Hide
          oflebbe Olaf Flebbe added a comment -

          Implement Change for CentOS

          Show
          oflebbe Olaf Flebbe added a comment - Implement Change for CentOS

            People

            • Assignee:
              oflebbe Olaf Flebbe
              Reporter:
              oflebbe Olaf Flebbe
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development