[ISSUE apache/dubbo#13862] Migrate dubbo-samples-triple-rest-openapi-generator to Spring Boot 3.2.3 - #1301
Conversation
…2.3 (#13862) - Bump spring-boot-starter-parent from 2.7.18 to 3.2.3, aligning the module with the dubbo-samples-triple-rest aggregate parent BOM; this was the last sample still on Spring Boot 2.x - Replace javax.annotation-api with jakarta.annotation-api and migrate the generated sources to jakarta.annotation.Generated - Enable useJakartaEe in the openapi-generator config so regenerated sources stay on the jakarta namespace - Update README system requirements to Java 17+ Verified with 'mvn clean package -DskipTests' (same as CI build opts): build succeeds on Spring Boot 3.2.3 / JDK 17.
Previous run failed on unrelated infra issues: - build-samples: Maven Central returned 502 while resolving spring-boot-starter-aop:2.3.12.RELEASE for dubbo-samples-spring-boot-hystrix - Integration Test (Java21, Job5): dubbo-samples-triple-rest-spring-security build failure against dubbo.version=3.3.7-SNAPSHOT
|
Friendly bump after two weeks: this migrates the last remaining Spring Boot 2 sample in |
|
Closing this for now: I've hit the per-user open pull request limit on my account and need the slot for a higher-priority change. The branch stays available on my fork, and I'm happy to reopen this once a slot frees up. Thanks! |
|
The Up to that point the shard was green (
Could someone with write access re-run the failed job? Happy to push a trivial retrigger instead if that's easier for the workflow. |
Migrates the last remaining Spring Boot 2 sample to Boot 3, as tracked in apache/dubbo#13862 (claim comment: apache/dubbo#13862 (comment)).
Changes
spring-boot-starter-parentfrom 2.7.18 to 3.2.3 indubbo-samples-triple-rest-openapi-generator, aligning the module with thedubbo-samples-triple-restaggregate parent BOM — this was the last sample in the repository still on Spring Boot 2.xjavax.annotation-apiwithjakarta.annotation-apiand migrate the generated sources fromjavax.annotation.Generatedtojakarta.annotation.GenerateduseJakartaEein thepom-generator.xmlopenapi-generator config (7.16.0 supports it) so future regenerations stay on the jakarta namespaceVerification
mvn clean package -DskipTestssucceeds on JDK 17 (same command as the CI build opts, which also test againstspring-boot.version:3.2.3)javaximports remain undersrc/Notes
contextLoadstest requires a running Zookeeper and is skipped by CI's-DskipTestsbuild opts, so it is not affected by this change.