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

          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.

          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

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

              Created:
              Updated:
              Resolved: