Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-54674

PCT: Add Docker packaging for Java 11

XMLWordPrintable

      After the experiments in https://github.com/jenkinsci/plugin-compat-tester/pull/84 is seems we will have to package two JDK versions into the Docker image. Building with JDK 11 will require a lot of patches on the plugin side.

      Advantage: with such approach we can have a single image for Java 8/11 and just switching the mode depending on the passed arguments

       

       

       

          [JENKINS-54674] PCT: Add Docker packaging for Java 11

          Adrien Lecharpentier added a comment -

          oleg_nenashev would it be acceptable to use something like getopts in the shell script to ease the configuration of the script? 

          This way, with one arguments, like --jdk11, we could set the required JVM and CLI parameters. I would also have to review the script a bit.

          Adrien Lecharpentier added a comment - oleg_nenashev would it be acceptable to use something like getopts in the shell script to ease the configuration of the script?  This way, with one arguments, like --jdk11 , we could set the required JVM and CLI parameters. I would also have to review the script a bit.

          Oleg Nenashev added a comment -

          Yes, it will acceptable imho

          Oleg Nenashev added a comment - Yes, it will acceptable imho

          Oleg Nenashev added a comment -

          Integrated into the master branch. Unfortnately there was no PCT releases for a long time, so we cannot just release it (CC rarabaolaza). But it's not required for Java 11 needs anyway

          Oleg Nenashev added a comment - Integrated into the master branch. Unfortnately there was no PCT releases for a long time, so we cannot just release it (CC rarabaolaza ). But it's not required for Java 11 needs anyway

          Oleg Nenashev added a comment -

          Reopening, because the patch caused issues with Java 8 in the image

          Oleg Nenashev added a comment - Reopening, because the patch caused issues with Java 8 in the image

          Oleg Nenashev added a comment -

          Hotfix was integrated: https://github.com/jenkinsci/plugin-compat-tester

           

          Oleg Nenashev added a comment - Hotfix was integrated: https://github.com/jenkinsci/plugin-compat-tester  

            alecharp Adrien Lecharpentier
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: