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

create provisioner test for trunk packages with jdk8

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: None
    • Labels:
      None

      Description

      This patch depends on BIGTOP-2661 , BIGTOP-2662, BIGTOP-2663.

      I was able to run the pi example with all these changes with current trunk on debian.

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        Is this going into 1.2?

        Show
        cos Konstantin Boudnik added a comment - Is this going into 1.2?
        Hide
        oflebbe Olaf Flebbe added a comment -

        Evans Ye: Could you please review ?

        Show
        oflebbe Olaf Flebbe added a comment - Evans Ye : Could you please review ?
        Hide
        evans_ye Evans Ye added a comment -

        If we're targeting at JDK 8 in 1.2.0, I think we should directly update all the java version to be JDK 8 instead of providing debian_jdk8 only. I can help to run through tests. How do you guys think?

        Show
        evans_ye Evans Ye added a comment - If we're targeting at JDK 8 in 1.2.0, I think we should directly update all the java version to be JDK 8 instead of providing debian_jdk8 only. I can help to run through tests. How do you guys think?
        Hide
        oflebbe Olaf Flebbe added a comment -

        The default should be jdk8, since we build with it.

        Show
        oflebbe Olaf Flebbe added a comment - The default should be jdk8, since we build with it.
        Hide
        evans_ye Evans Ye added a comment -

        I mean we should upgrade all the jdk to be 8 in all the provided config*.yaml, that way the runtime env aligns what we built.

        Show
        evans_ye Evans Ye added a comment - I mean we should upgrade all the jdk to be 8 in all the provided config*.yaml, that way the runtime env aligns what we built.
        Hide
        cos Konstantin Boudnik added a comment -

        +1

        Show
        cos Konstantin Boudnik added a comment - +1
        Hide
        oflebbe Olaf Flebbe added a comment -

        Feel free to propose a solution. Removed the assignment

        Show
        oflebbe Olaf Flebbe added a comment - Feel free to propose a solution. Removed the assignment
        Hide
        rvs Roman Shaposhnik added a comment -

        How about this patch?

        Show
        rvs Roman Shaposhnik added a comment - How about this patch?
        Hide
        cos Konstantin Boudnik added a comment -

        Looks about right without testing. Perhaps Evans Ye and Olaf Flebbe could chime in as well

        Show
        cos Konstantin Boudnik added a comment - Looks about right without testing. Perhaps Evans Ye and Olaf Flebbe could chime in as well
        Hide
        oflebbe Olaf Flebbe added a comment -

        Hm... not tested but looked quickly at it. Do we actually need

        jdk: "java-1.8.0-openjdk-devel.x86_64"

        wouldn't

        jdk: "java-1.8.0-openjdk-devel"

        be sufficient and more portable, arch-wise.

        Show
        oflebbe Olaf Flebbe added a comment - Hm... not tested but looked quickly at it. Do we actually need jdk: "java-1.8.0-openjdk-devel.x86_64" wouldn't jdk: "java-1.8.0-openjdk-devel" be sufficient and more portable, arch-wise.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development