• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • core

      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.

          [JENKINS-24827] maven-hpi-plugin should be upgraded to Jetty 8

          Ryan Campbell added a comment -

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

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

          Ryan Campbell added a comment -

          Ryan Campbell added a comment - Discussion: https://groups.google.com/forum/#!topic/jenkinsci-dev/QCGogzAeP4Q

          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.

          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.

          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

          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

          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

          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

          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

          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

            vlatombe Vincent Latombe
            recampbell Ryan Campbell
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: