Skip to content

fix(internal/serviceconfig): port release_level info for java from generation_config.yaml#4862

Merged
zhumin8 merged 3 commits intogoogleapis:mainfrom
zhumin8:java-migarte-release-level
Mar 27, 2026
Merged

fix(internal/serviceconfig): port release_level info for java from generation_config.yaml#4862
zhumin8 merged 3 commits intogoogleapis:mainfrom
zhumin8:java-migarte-release-level

Conversation

@zhumin8
Copy link
Copy Markdown
Contributor

@zhumin8 zhumin8 commented Mar 26, 2026

Ports over release_level info from google-cloud-java/generation_config.yaml to sdk.yaml. Filled out "preview" for empty values (default in java's current hermetic build logic), removed "stable" as it is default.
Changes are made by running a temporarily script (included in commit 957dceb for record sake).

go run tool/cmd/java_release_level_migrate/main.go --java-repo <path-to-google-cloud-java-repo>

This is a fix after #4835 moved the source of truth for release levels to sdk.yaml.

Fix #4859

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the internal/serviceconfig/sdk.yaml file to add java: preview to the release_level configuration for numerous service paths. The feedback indicates that the update is currently incomplete, as several service paths that support Java were missed and still lack the explicit java: preview entry in their release_level blocks.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.38%. Comparing base (cec5d8b) to head (e191a00).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4862   +/-   ##
=======================================
  Coverage   81.38%   81.38%           
=======================================
  Files         130      130           
  Lines       10910    10910           
=======================================
  Hits         8879     8879           
  Misses       1392     1392           
  Partials      639      639           

☔ View full report in Codecov by Sentry.
📢 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.

@zhumin8 zhumin8 marked this pull request as ready for review March 26, 2026 21:34
@zhumin8 zhumin8 requested a review from a team as a code owner March 26, 2026 21:34
@zhumin8 zhumin8 enabled auto-merge (squash) March 26, 2026 21:34
Copy link
Copy Markdown
Contributor

@JoeWang1127 JoeWang1127 left a comment

Choose a reason for hiding this comment

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

Could you add the command in the PR description.

@zhumin8 zhumin8 merged commit 648ed1d into googleapis:main Mar 27, 2026
26 of 28 checks passed
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.

java: fix release level differences

2 participants