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

Investigate Impact of java.sql modularization on classic Jenkins modes

    Details

    • Similar Issues:
    • Released As:
      Jenkinsfile Runner 1.0-beta-6

      Description

      I discovered that Jenkinsfile Runner does not work reliably with Java 11 when there is a plugin using "java.sql" internally (JENKINS-54426). It impacts Pipeline Utility Steps (JENKINS-54355), and it will likely also impact Pipeline: Maven Plugin and other plugins: https://github.com/search?p=2&q=org%3Ajenkinsci+%22java.sql%22&type=Code . For JFR the most critical thing is Pipeline plugins, of course.  CC rsandell Cyrille Le Clerc

      We need to investigate whether the failing behavior is also possible on classic Jenkins instances

       

        Attachments

          Issue Links

            Activity

            Hide
            batmat Baptiste Mathus added a comment - - edited

            > We need to investigate whether the failing behavior is also possible on classic Jenkins instances

            This has been analyzed by Adrien Lecharpentier in JENKINS-54355 and it is not reproducible in a standard environment. Pipeline-utility-steps works perfectly fine on JDK11 without using any additional --add-modules ...,java.sql switch.

            We've seen this issue surface at least in PCT, in JFR, so my current suspicion is that this issue is triggered by a special classloading used there (JTH?).

            Show
            batmat Baptiste Mathus added a comment - - edited > We need to investigate whether the failing behavior is also possible on classic Jenkins instances This has been analyzed by Adrien Lecharpentier in JENKINS-54355 and it is not reproducible in a standard environment. Pipeline-utility-steps works perfectly fine on JDK11 without using any additional --add-modules ...,java.sql switch. We've seen this issue surface at least in PCT, in JFR, so my current suspicion is that this issue is triggered by a special classloading used there (JTH?).
            Hide
            vilacides Isa Vilacides added a comment -

            Since Francisco Fernández removed junit he is the best suited to try to reproduce this issue cause it's likely that it's not happening anymore

            Show
            vilacides Isa Vilacides added a comment - Since Francisco Fernández removed junit he is the best suited to try to reproduce this issue cause it's likely that it's not happening anymore
            Hide
            oleg_nenashev Oleg Nenashev added a comment -
            Show
            oleg_nenashev Oleg Nenashev added a comment - I believe it was fixed by Evaristo Gutierrez in https://github.com/jenkinsci/jenkinsfile-runner/commits/master   If Evaristo Gutierrez and Francisco Fernández agree, I propose to close it
            Hide
            egutierrez Evaristo Gutierrez added a comment -

            Sounds good to me. It should be solved as proved by https://github.com/jenkinsci/jenkinsfile-runner/tree/master/demo/cwp-jdk11

            Show
            egutierrez Evaristo Gutierrez added a comment - Sounds good to me. It should be solved as proved by  https://github.com/jenkinsci/jenkinsfile-runner/tree/master/demo/cwp-jdk11
            Hide
            fcojfernandez Francisco Fernández added a comment -

            Go ahead Oleg! I'm fine with closing it

            Show
            fcojfernandez Francisco Fernández added a comment - Go ahead Oleg! I'm fine with closing it

              People

              • Assignee:
                Unassigned
                Reporter:
                oleg_nenashev Oleg Nenashev
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: