feat: add OpenRewrite module for migrations and recipe for 5.3.0#3236
Conversation
Sorry, something went wrong.
There was a problem hiding this comment.
Pull request overview
This PR adds a new migration module to the Java Operator SDK project, providing OpenRewrite recipes to automate migration from v5.2 to v5.3. The recipes handle dependency version upgrades, artifact renames, Metrics API method renames, and removal of deprecated method invocations.
Changes:
- New
migrationMaven module with OpenRewrite recipe definitions for v5.2→v5.3 migration (JUnit module rename, Metrics method renames, version upgrade,monitorSizeOfremoval) - Test class
V53MigrationTestvalidating the Maven artifact rename and Metrics method rename recipes - Documentation update adding OpenRewrite usage instructions to the v5.3 migration guide
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Adds the new migration module to the reactor build |
| migration/pom.xml | New module POM with OpenRewrite dependencies for recipe authoring and testing |
| migration/src/main/resources/META-INF/rewrite/v5-3-migration.yml | OpenRewrite recipe definitions for v5.3 migration |
| migration/src/test/java/.../V53MigrationTest.java | Tests for the JUnit module rename and Metrics method rename recipes |
| docs/content/en/docs/migration/v5-3-migration.md | Adds OpenRewrite automated migration instructions |
You can also share your feedback on Copilot code review. Take the survey.
Sorry, something went wrong.
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
|
added additional improvements, but I think we should stop there, this already adresses most of the issues; there are some others but those are quite deep rabit holes |
Sorry, something went wrong.
xstefank
left a comment
xstefank
left a comment
There was a problem hiding this comment.
I just wonder if we can't reuse some of the existing openrewrite recipes.
Sorry, something went wrong.
What do you mean? If you take a look we are reusing a lot of them, there is just one case that current recipies not cover, removing the method from classes that implement an interface... but yeah I did not found for that case any... |
Sorry, something went wrong.
csviri
merged commit b240aef
into
operator-framework:main
Mar 18, 2026
Reviewers
Copilot code review Copilot Copilot left review comments
No description provided.