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

maven-hpi-plugin should be upgraded to Jetty 8

    Details

    • Type: Improvement
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
    • Similar Issues:

      Description

      hpi:run currently starts Jetty 6, whereas the default container for jenkins.war is Jetty 8. So we should upgrade hpi:run to launch a Jetty 8 container.

        Attachments

          Issue Links

            Activity

            Hide
            recampbell Ryan Campbell added a comment -

            I was able to get this partially complete here: https://github.com/jenkinsci/maven-hpi-plugin/pull/11

            Show
            recampbell Ryan Campbell added a comment - I was able to get this partially complete here: https://github.com/jenkinsci/maven-hpi-plugin/pull/11
            Show
            recampbell Ryan Campbell added a comment - Discussion: https://groups.google.com/forum/#!topic/jenkinsci-dev/QCGogzAeP4Q
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Ryan Campbell
            Path:
            pom.xml
            src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJetty6Mojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJettyMojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/ConsoleScanner.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/RunMojo.java
            http://jenkins-ci.org/commit/maven-hpi-plugin/2634db75512389770f65dce77af59808610ef562
            Log:
            JENKINS-24827 Port maven-hpi-plugin to Jetty 8. Partially complete – doesn't boot correctly, perhaps due to classpath issues. See JENKINS-24827 for more information.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ryan Campbell Path: pom.xml src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJetty6Mojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJettyMojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/ConsoleScanner.java src/main/java/org/jenkinsci/maven/plugins/hpi/RunMojo.java http://jenkins-ci.org/commit/maven-hpi-plugin/2634db75512389770f65dce77af59808610ef562 Log: JENKINS-24827 Port maven-hpi-plugin to Jetty 8. Partially complete – doesn't boot correctly, perhaps due to classpath issues. See JENKINS-24827 for more information.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent LATOMBE
            Path:
            pom.xml
            src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJettyMojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/JettyAndServletApiOnlyClassLoader.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/RunMojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/ServletApiOnlyClassLoader.java
            http://jenkins-ci.org/commit/maven-hpi-plugin/b93b423deeedca95e6f0565723802c0bb81b4472
            Log:
            JENKINS-24827 Complete compatibility with jetty 8.x

            • Update jetty to latest 8.x
            • Expose jetty packages to webapp as it is required for jndi initial
              factory
            • Rename org.mortbay.jetty to org.eclipse.jetty
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent LATOMBE Path: pom.xml src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJettyMojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/JettyAndServletApiOnlyClassLoader.java src/main/java/org/jenkinsci/maven/plugins/hpi/RunMojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/ServletApiOnlyClassLoader.java http://jenkins-ci.org/commit/maven-hpi-plugin/b93b423deeedca95e6f0565723802c0bb81b4472 Log: JENKINS-24827 Complete compatibility with jetty 8.x Update jetty to latest 8.x Expose jetty packages to webapp as it is required for jndi initial factory Rename org.mortbay.jetty to org.eclipse.jetty
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            pom.xml
            src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJetty6Mojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJettyMojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/ConsoleScanner.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/JettyAndServletApiOnlyClassLoader.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/RunMojo.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/ServletApiOnlyClassLoader.java
            http://jenkins-ci.org/commit/maven-hpi-plugin/b2d139e78da68c5d05edce9925480d0e7b21ffdf
            Log:
            Merge pull request #12 from Vlatombe/JENKINS-24827

            JENKINS-24827 Port hpi:run to jetty 8.x

            Compare: https://github.com/jenkinsci/maven-hpi-plugin/compare/f361e8e1b4b7...b2d139e78da6

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: pom.xml src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJetty6Mojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/AbstractJettyMojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/ConsoleScanner.java src/main/java/org/jenkinsci/maven/plugins/hpi/JettyAndServletApiOnlyClassLoader.java src/main/java/org/jenkinsci/maven/plugins/hpi/RunMojo.java src/main/java/org/jenkinsci/maven/plugins/hpi/ServletApiOnlyClassLoader.java http://jenkins-ci.org/commit/maven-hpi-plugin/b2d139e78da68c5d05edce9925480d0e7b21ffdf Log: Merge pull request #12 from Vlatombe/ JENKINS-24827 JENKINS-24827 Port hpi:run to jetty 8.x Compare: https://github.com/jenkinsci/maven-hpi-plugin/compare/f361e8e1b4b7...b2d139e78da6
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            pom.xml
            src/main/java/org/jenkinsci/maven/plugins/hpi/CollectionUtils.java
            src/main/java/org/jenkinsci/maven/plugins/hpi/JettyAndServletApiOnlyClassLoader.java
            http://jenkins-ci.org/commit/maven-hpi-plugin/68fbbd106a1a863f79a48e3fca734230ccf5a67b
            Log:
            Merge pull request #19 from Vlatombe/JENKINS-24827

            Restore jdk6 compatibility

            Compare: https://github.com/jenkinsci/maven-hpi-plugin/compare/78247d32be98...68fbbd106a1a

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: pom.xml src/main/java/org/jenkinsci/maven/plugins/hpi/CollectionUtils.java src/main/java/org/jenkinsci/maven/plugins/hpi/JettyAndServletApiOnlyClassLoader.java http://jenkins-ci.org/commit/maven-hpi-plugin/68fbbd106a1a863f79a48e3fca734230ccf5a67b Log: Merge pull request #19 from Vlatombe/ JENKINS-24827 Restore jdk6 compatibility Compare: https://github.com/jenkinsci/maven-hpi-plugin/compare/78247d32be98...68fbbd106a1a

              People

              • Assignee:
                vlatombe Vincent Latombe
                Reporter:
                recampbell Ryan Campbell
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: