Skip to content

Conversation

@klopfdreh
Copy link

@klopfdreh klopfdreh commented Jan 30, 2026

Description

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change
  • Chore (non-breaking change which doesn't affect codebase;
    test, version modification, documentation, etc.)

Checklist

  • Code contributed by me aligns with current project license: Apache 2.0
  • I Added CHANGELOG entry regarding this change
  • I have implemented unit tests to cover my changes
  • I have added/updated the javadocs and other documentation accordingly
  • No new bugs, code smells, etc. in SonarCloud report
  • I tested my code in Kubernetes
  • I tested my code in OpenShift

Issues Ref: #7421

@klopfdreh
Copy link
Author

klopfdreh commented Jan 30, 2026

  • E2E Tests / BuildWithoutTests failed because package okhttp3 does not exist
  • Java 17 Maven (pull_request) failed because of kubernetes-httpclient-jdk which was not affected by the change
  • Java 21 Maven (pull_request) failed because of the same cause like E2E Tests / BuildWithoutTests
  • Java 11 Maven (pull_request) failed because of the same cause like E2E Tests / BuildWithoutTests

I added okhttp-jvm manually to kuberntes-examples as there was no bom import for okhttp - a new feature of version 5.x is that the environment is discovered automatically and okhttp-jvm or okhttp-android is added as dependency. This is somehow not working in the github action. But when I used the workaround described in the issue okhttp-jvm is pulled like it should be.

@klopfdreh klopfdreh force-pushed the feature/okhttpupgrade branch from 40bd5ec to 7bbcab1 Compare January 30, 2026 06:42
@klopfdreh
Copy link
Author

All right - all checks passed. 💚 😃

@ash-thakur-rh
Copy link
Member

Hi @klopfdreh , Thank you for the PR! We'll have a look into it soon and will try to include your changes.

@klopfdreh klopfdreh force-pushed the feature/okhttpupgrade branch from 7bbcab1 to 1a70125 Compare February 3, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants