Skip to content

CTM-587: downgrade client-java back to 24.0.0 to avoid issues with okhttp - #1754

Merged
davidangb merged 1 commit into
developfrom
da_CTM-587_revert_client_java
Jul 28, 2026
Merged

davidangb merged 1 commit into
developfrom
da_CTM-587_revert_client_java

Conversation

@davidangb

@davidangb davidangb commented Jul 27, 2026 •

Copy link
Copy Markdown
Contributor

Downgrades io.kubernetes:client-java to the most recent version that still uses okhttp 4.

As of 25.0.0, client-java pulls in okhttp 5. We have other important dependencies, like terra-common-lib, that pull in okhttp 4. Versions 4 and 5 can't coexist, and cause problems at build time if both are present. We're seeing those problems in:

So, this PR downgrades client-java to the last-good version. Note this is still one version higher than where we were two weeks ago; we only tried moving to the later version of client-java in #1752.

I manually published this branch, as version 8e8c086e-SNAP, and saw that it fixed the sbt assembly issues in Sam and Rawls.


PR checklist

  • For each library you've modified here, decide whether it requires a major, minor, or no version bump. (Click here for guidance)

If you're doing a major or minor version bump to any libraries:

  • Bump the version in project/Settings.scala createVersion()
  • Update CHANGELOG.md for those libraries
  • I promise I used @deprecated instead of deleting code where possible

In all cases:

  • Replace the appropriate version hashes in README.md and the CHANGELOG.md for any libs you changed with TRAVIS-REPLACE-ME to auto-update the version string
  • Get two thumbsworth of PR review
  • Verify all tests go green (CI and coverage tests)
  • Squash commits and merge to develop
  • Delete branch after merge

@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 25.43%. Comparing base (adcc4d4) to head (8e8c086).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1754   +/-   ##
========================================
  Coverage    25.43%   25.43%           
========================================
  Files          105      105           
  Lines         2693     2693           
  Branches       251      251           
========================================
  Hits           685      685           
  Misses        2008     2008           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@davidangb
davidangb marked this pull request as ready for review July 27, 2026 20:31
@davidangb
davidangb merged commit fccb671 into develop Jul 28, 2026
5 checks passed
@davidangb
davidangb deleted the da_CTM-587_revert_client_java branch July 28, 2026 13:12
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