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

Adapt Support Core plugin to EE 9

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Unresolved
    • Icon: Major Major
    • support-core-plugin
    • None

      In src/main/java/com/cloudbees/jenkins/support/impl/AboutJenkins.java, support-core uses a Stapler API (Stapler.getCurrent().getServletContext()) that will change incompatibly between EE 8 and EE 9 in JENKINS-73258. https://github.com/jenkinsci/support-core-plugin/pull/543 "prepared" the plugin in a hackish way by simply ignoring the failure on EE 9, since this was not critical functionality. But at some point in time, adoption will shift such that EE 9 is used more than EE 8, and at that point it won't make sense to keep supporting EE 8 at the expense of EE 9. At that point in time, https://github.com/jenkinsci/support-core-plugin/pull/543 should be reverted, and to get the plugin to compile again the javax.servlet imports should be changed to jakarta.servlet imports. Note that this will require bumping the plugin's jenkins.version to a baseline that includes EE 9, something that the support-core maintainers might not want to do until EE 9 ships in LTS, or possibly even several LTS releases.

      Since https://github.com/jenkinsci/support-core-plugin/pull/543 "prepared" the plugin (by disabling the feature on EE 9), this change does not have to be done in lockstep with JENKINS-73278 (unlike JENKINS-73279, which does need to be done in lockstep). The guiding factor for when to perform this change should be the support-core plugin maintainers' policy for dropping support for older cores. As soon as they are ready to drop support for EE 8 (which may not be for several LTS releases after EE 9 ships), then this ticket should be prioritized.

      CC allan_burdajewicz

            Unassigned Unassigned
            basil Basil Crow
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: