Skip to content

[SPARK-52086] Upgrade operator-sdk to 5.1.1 #252

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

csviri
Copy link

@csviri csviri commented Jun 18, 2025

What changes were proposed in this pull request?

Change Java Operator SDK version to v5.1

The intention is to keep the functionality the same as the previous version;

Does this PR introduce any user-facing change?

No

How was this patch tested?

Standard CI should test it.

Was this patch authored or co-authored using generative AI tooling?

No

@@ -15,9 +15,9 @@
# specific language governing permissions and limitations
# under the License.
[versions]
fabric8 = "7.3.0"
fabric8 = "7.3.1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for making a PR, but please revert this, @csviri . You can do this upgrade independently.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ur, wait, your PR looks strange. We already have this.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh, yes, sorry, did this rather hastily. Rebased now.

@dongjoon-hyun
Copy link
Member

cc @jiangzho , too

@dongjoon-hyun dongjoon-hyun changed the title Bump Java Operator SDK to v5.1 [SPARK-52086] Upgrade operator-sdk to v5.1 Jun 18, 2025
@dongjoon-hyun dongjoon-hyun changed the title [SPARK-52086] Upgrade operator-sdk to v5.1 [SPARK-52086] Upgrade operator-sdk to v5.1.1 Jun 18, 2025
@dongjoon-hyun dongjoon-hyun changed the title [SPARK-52086] Upgrade operator-sdk to v5.1.1 [SPARK-52086] Upgrade operator-sdk to 5.1.1 Jun 18, 2025
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase your branch and make the CI happy, @csviri .

csviri added 2 commits June 19, 2025 11:13
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@dongjoon-hyun
Copy link
Member

Thank you for updating, @csviri .

@dongjoon-hyun
Copy link
Member

It seems to fail still. Could you take a look at the failure, @csviri ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants