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

PCT not taking into account jdk version for detached plugins

    • PCT 0.1.0

      Since JENKINS-55681 splits can be JVM version dependent, but the PCT is not able to detect the JDK version field on the splits file and is adding those plugins always. As a result the PCT is incorrectly adding `jaxb` plugin even to java 8 runs

          [JENKINS-56312] PCT not taking into account jdk version for detached plugins

          Raul Arabaolaza created issue -
          Raul Arabaolaza made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Raul Arabaolaza made changes -
          Summary Original: PCT not taking into account New: PCT not taking into account jdk version for detached plugins
          Raul Arabaolaza made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Baptiste Mathus made changes -
          Remote Link New: This issue links to "PR (Web Link)" [ 22745 ]

          FTR, the PR https://github.com/jenkinsci/plugin-compat-tester/pull/115 seems to have introduced a regression. We confirmed it while working on adding more Integration tests through https://github.com/jenkinsci/plugin-compat-tester/pull/140 (this PR is currently red, when it should be building fine).

          mramonleon is going to file a report about that just now.

          Baptiste Mathus added a comment - FTR, the PR https://github.com/jenkinsci/plugin-compat-tester/pull/115 seems to have introduced a regression. We confirmed it while working on adding more Integration tests through https://github.com/jenkinsci/plugin-compat-tester/pull/140 (this PR is currently red, when it should be building fine). mramonleon is going to file a report about that just now.
          Ramon Leon made changes -
          Link New: This issue causes JENKINS-57320 [ JENKINS-57320 ]

          Oleg Nenashev added a comment -

          There was a bug in the implementation, fixed in https://github.com/jenkinsci/plugin-compat-tester/pull/140/commits/d6ce90e1c90a2db0f4b934ccc2784b66a26e8298 

          But JENKINS-57320  was introduced by another issue

          Oleg Nenashev added a comment - There was a bug in the implementation, fixed in https://github.com/jenkinsci/plugin-compat-tester/pull/140/commits/d6ce90e1c90a2db0f4b934ccc2784b66a26e8298   But  JENKINS-57320   was introduced by another issue

          Oleg Nenashev added a comment -

          Oleg Nenashev added a comment - https://github.com/jenkinsci/plugin-compat-tester/releases/tag/plugins-compat-tester-aggregator-0.1.0
          Oleg Nenashev made changes -
          Released As New: PCT 0.1.0
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Resolved [ 5 ]

            rarabaolaza Raul Arabaolaza
            rarabaolaza Raul Arabaolaza
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: