Migrate from EE 8 to EE 9

jenkinsci · Jan 16, 2025 · b86bb78 · b86bb78
1 parent 1fb1ac6
commit b86bb78
Show file tree

Hide file tree

Showing 9 changed files with 33 additions and 32 deletions.
diff --git a/Jenkinsfile b/Jenkinsfile
@@ -8,5 +8,4 @@ buildPlugin(
   configurations: [
     [platform: 'linux', jdk: 21],
     [platform: 'windows', jdk: 17],
-    [platform: 'linux', jdk: 11],
 ])
diff --git a/...ine-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ASTSchema.java b/...ine-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ASTSchema.java
@@ -29,10 +29,10 @@
 import com.github.fge.jsonschema.main.JsonSchemaFactory;
 import hudson.Extension;
 import hudson.model.RootAction;
-import org.kohsuke.stapler.StaplerRequest;
-import org.kohsuke.stapler.StaplerResponse;
+import org.kohsuke.stapler.StaplerRequest2;
+import org.kohsuke.stapler.StaplerResponse2;
 
-import javax.servlet.ServletException;
+import jakarta.servlet.ServletException;
 import java.io.IOException;
 
 /**
@@ -60,7 +60,7 @@ public String getDisplayName() {
     }
 
     @SuppressWarnings("unused")
-    public void doJson(StaplerRequest req, StaplerResponse rsp) throws IOException, ServletException {
+    public void doJson(StaplerRequest2 req, StaplerResponse2 rsp) throws IOException, ServletException {
         rsp.serveFile(req, getClass().getResource("/ast-schema.json"));
     }
 

diff --git a/.../jenkinsci/plugins/pipeline/modeldefinition/actions/RestartDeclarativePipelineAction.java b/.../jenkinsci/plugins/pipeline/modeldefinition/actions/RestartDeclarativePipelineAction.java
@@ -50,15 +50,15 @@
 import org.kohsuke.accmod.restrictions.NoExternalUse;
 import org.kohsuke.stapler.HttpResponse;
 import org.kohsuke.stapler.QueryParameter;
-import org.kohsuke.stapler.StaplerRequest;
-import org.kohsuke.stapler.StaplerResponse;
+import org.kohsuke.stapler.StaplerRequest2;
+import org.kohsuke.stapler.StaplerResponse2;
 import org.kohsuke.stapler.export.Exported;
 import org.kohsuke.stapler.export.ExportedBean;
 import org.kohsuke.stapler.interceptor.RequirePOST;
 
 import edu.umd.cs.findbugs.annotations.CheckForNull;
 import edu.umd.cs.findbugs.annotations.NonNull;
-import javax.servlet.ServletException;
+import jakarta.servlet.ServletException;
 import java.io.IOException;
 import java.util.*;
 
@@ -137,7 +137,7 @@ public HttpResponse doRestartPipeline(@QueryParameter String stageName) {
 
     @Restricted(DoNotUse.class)
     @RequirePOST
-    public void doRestart(StaplerRequest req, StaplerResponse rsp) throws ServletException, IOException {
+    public void doRestart(StaplerRequest2 req, StaplerResponse2 rsp) throws ServletException, IOException {
         if (!isRestartEnabled()) {
             throw new AccessDeniedException("not allowed to restart"); // AccessDeniedException2 requires us to look up the specific Permission
         }

diff --git a/...n/java/org/jenkinsci/plugins/pipeline/modeldefinition/endpoints/ModelConverterAction.java b/...n/java/org/jenkinsci/plugins/pipeline/modeldefinition/endpoints/ModelConverterAction.java
@@ -43,13 +43,13 @@
 import org.jenkinsci.plugins.pipeline.modeldefinition.parser.JSONParser;
 import org.jenkinsci.plugins.pipeline.modeldefinition.validator.ErrorCollector;
 import org.kohsuke.stapler.HttpResponse;
-import org.kohsuke.stapler.StaplerRequest;
+import org.kohsuke.stapler.StaplerRequest2;
 import org.kohsuke.stapler.interceptor.RequirePOST;
 
-import javax.servlet.FilterChain;
-import javax.servlet.ServletException;
-import javax.servlet.http.HttpServletRequest;
-import javax.servlet.http.HttpServletResponse;
+import jakarta.servlet.FilterChain;
+import jakarta.servlet.ServletException;
+import jakarta.servlet.http.HttpServletRequest;
+import jakarta.servlet.http.HttpServletResponse;
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.io.StringWriter;
@@ -82,7 +82,7 @@ public String getDisplayName() {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doToJenkinsfile(StaplerRequest req) {
+    public HttpResponse doToJenkinsfile(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         JSONObject result = new JSONObject();
@@ -122,7 +122,7 @@ public HttpResponse doToJenkinsfile(StaplerRequest req) {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doToJson(StaplerRequest req) {
+    public HttpResponse doToJson(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         JSONObject result = new JSONObject();
@@ -150,7 +150,7 @@ public HttpResponse doToJson(StaplerRequest req) {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doStepsToJson(StaplerRequest req) {
+    public HttpResponse doStepsToJson(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         JSONObject result = new JSONObject();
@@ -178,7 +178,7 @@ public HttpResponse doStepsToJson(StaplerRequest req) {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doStepsToJenkinsfile(StaplerRequest req) {
+    public HttpResponse doStepsToJenkinsfile(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         JSONObject result = new JSONObject();
@@ -234,7 +234,7 @@ public HttpResponse doStepsToJenkinsfile(StaplerRequest req) {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doValidateJenkinsfile(StaplerRequest req) {
+    public HttpResponse doValidateJenkinsfile(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         JSONObject result = new JSONObject();
@@ -262,7 +262,7 @@ public HttpResponse doValidateJenkinsfile(StaplerRequest req) {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doValidateJson(StaplerRequest req) {
+    public HttpResponse doValidateJson(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         JSONObject result = new JSONObject();
@@ -304,7 +304,7 @@ public HttpResponse doValidateJson(StaplerRequest req) {
 
     @SuppressWarnings("unused")
     @RequirePOST
-    public HttpResponse doValidate(StaplerRequest req) {
+    public HttpResponse doValidate(StaplerRequest2 req) {
         Jenkins.get().checkPermission(Jenkins.READ);
 
         List<String> output = new ArrayList<>();

diff --git a/...ain/java/org/jenkinsci/plugins/pipeline/modeldefinition/generator/DirectiveGenerator.java b/...ain/java/org/jenkinsci/plugins/pipeline/modeldefinition/generator/DirectiveGenerator.java
@@ -48,7 +48,7 @@
 import org.kohsuke.stapler.HttpResponse;
 import org.kohsuke.stapler.HttpResponses;
 import org.kohsuke.stapler.QueryParameter;
-import org.kohsuke.stapler.StaplerRequest;
+import org.kohsuke.stapler.StaplerRequest2;
 
 import edu.umd.cs.findbugs.annotations.NonNull;
 import java.util.*;
@@ -75,7 +75,7 @@ public List<DirectiveDescriptor> getDirectives() {
     }
 
     @Restricted(DoNotUse.class) // accessed via REST API
-    public HttpResponse doGenerateDirective(StaplerRequest req, @QueryParameter String json) throws Exception {
+    public HttpResponse doGenerateDirective(StaplerRequest2 req, @QueryParameter String json) throws Exception {
         // TODO is there not an easier way to do this? Maybe Descriptor.newInstancesFromHeteroList on a one-element JSONArray?
         JSONObject jsonO = JSONObject.fromObject(json);
         Jenkins j = Jenkins.get();

diff --git a/...kinsci/plugins/pipeline/modeldefinition/actions/RestartDeclarativePipelineActionTest.java b/...kinsci/plugins/pipeline/modeldefinition/actions/RestartDeclarativePipelineActionTest.java
@@ -41,6 +41,7 @@
 import hudson.security.SecurityRealm;
 import hudson.security.csrf.CrumbIssuer;
 import hudson.util.RunList;
+import jakarta.servlet.ServletRequest;
 import jenkins.branch.BranchSource;
 import jenkins.model.Jenkins;
 import jenkins.plugins.git.GitSCMSource;
@@ -788,7 +789,7 @@ public void testDoRestartPipeline() throws Exception {
 
         CrumbIssuer crumbIssuer = j.jenkins.getCrumbIssuer();
         if(crumbIssuer != null) {
-            params.add(new NameValuePair(crumbIssuer.getDescriptor().getCrumbRequestField(), crumbIssuer.getCrumb(null)));
+            params.add(new NameValuePair(crumbIssuer.getDescriptor().getCrumbRequestField(), crumbIssuer.getCrumb((ServletRequest) null)));
         }
 
         request.setRequestParameters(params);

diff --git a/...g/jenkinsci/plugins/pipeline/modeldefinition/endpoints/ModelConverterActionStepsTest.java b/...g/jenkinsci/plugins/pipeline/modeldefinition/endpoints/ModelConverterActionStepsTest.java
@@ -31,7 +31,7 @@
 import org.jenkinsci.plugins.pipeline.modeldefinition.AbstractModelDefTest;
 import org.junit.Test;
 import org.jvnet.hudson.test.JenkinsRule;
-import org.kohsuke.stapler.StaplerRequest;
+import org.kohsuke.stapler.StaplerRequest2;
 
 import java.io.IOException;
 import java.net.URL;
@@ -50,8 +50,8 @@
  * Tests steps conversion in {@link ModelConverterAction}
  *
  * @author Robert Sandell &lt;rsandell@cloudbees.com&gt;.
- * @see ModelConverterAction#doStepsToJenkinsfile(StaplerRequest)
- * @see ModelConverterAction#doStepsToJson(StaplerRequest)
+ * @see ModelConverterAction#doStepsToJenkinsfile(StaplerRequest2)
+ * @see ModelConverterAction#doStepsToJson(StaplerRequest2)
  */
 public class ModelConverterActionStepsTest extends AbstractModelDefTest {
 

diff --git a/...java/org/jenkinsci/plugins/pipeline/modeldefinition/generator/DirectiveGeneratorTest.java b/...java/org/jenkinsci/plugins/pipeline/modeldefinition/generator/DirectiveGeneratorTest.java
@@ -39,6 +39,7 @@
 import hudson.triggers.SCMTrigger;
 import hudson.triggers.TimerTrigger;
 import hudson.triggers.Trigger;
+import jakarta.servlet.ServletRequest;
 import jenkins.model.BuildDiscarderProperty;
 import jenkins.model.OptionalJobProperty;
 import net.sf.json.JSONArray;
@@ -650,7 +651,7 @@ private void assertGenerateDirective(@NonNull AbstractDirective desc, @NonNull S
         List<NameValuePair> params = new ArrayList<NameValuePair>();
         params.add(new NameValuePair("json", staplerJsonForDescr(desc).toString()));
         // WebClient.addCrumb *replaces* rather than *adds*:
-        params.add(new NameValuePair(r.jenkins.getCrumbIssuer().getDescriptor().getCrumbRequestField(), r.jenkins.getCrumbIssuer().getCrumb(null)));
+        params.add(new NameValuePair(r.jenkins.getCrumbIssuer().getDescriptor().getCrumbRequestField(), r.jenkins.getCrumbIssuer().getCrumb((ServletRequest) (ServletRequest) (ServletRequest) (ServletRequest) (ServletRequest) (ServletRequest) (ServletRequest) (ServletRequest) (ServletRequest) null)));
         wrs.setRequestParameters(params);
         WebResponse response = wc.getPage(wrs).getWebResponse();
         assertEquals("text/plain", response.getContentType());

diff --git a/pom.xml b/pom.xml
@@ -27,7 +27,7 @@
   <parent>
     <groupId>org.jenkins-ci.plugins</groupId>
     <artifactId>plugin</artifactId>
-    <version>4.88</version>
+    <version>5.5</version>
     <relativePath />
   </parent>
 
@@ -75,7 +75,7 @@
       <dependency>
         <groupId>io.jenkins.tools.bom</groupId>
         <artifactId>bom-${jenkins.baseline}.x</artifactId>
-        <version>3358.vea_fa_1f41504d</version>
+        <version>3850.vb_c5319efa_e29</version>
         <scope>import</scope>
         <type>pom</type>
       </dependency>
@@ -133,8 +133,8 @@
   <properties>
     <changelist>999999-SNAPSHOT</changelist>
     <gitHubRepo>jenkinsci/pipeline-model-definition-plugin</gitHubRepo>
-    <jenkins.baseline>2.440</jenkins.baseline>
-    <jenkins.version>${jenkins.baseline}.3</jenkins.version>
+    <jenkins.baseline>2.479</jenkins.baseline>
+    <jenkins.version>${jenkins.baseline}.1</jenkins.version>
   </properties>
 
   <profiles>