Skip to content

test: Release please simulation testing #12848#12849

Draft
meltsufin wants to merge 1 commit intorelease-please--fix-12825from
release-please--simulation
Draft

test: Release please simulation testing #12848#12849
meltsufin wants to merge 1 commit intorelease-please--fix-12825from
release-please--simulation

Conversation

@meltsufin
Copy link
Copy Markdown
Member

Test fallback branch in #12848

This commit updates the image_tag to a non-existent version (9.99.9) in hermetic_library_generation.yaml to force the pull failure and verify the fallback logic works in CI.

DO NOT MERGE THIS PR
@meltsufin meltsufin added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 18, 2026
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 introduces a fallback mechanism for Docker image tags in the hermetic library generation script, specifically targeting release branches. When a Docker image for the current tag is not found, the script now attempts to retrieve and use the image tag from the target branch. A review comment correctly identified that the script's 'pipefail' setting could cause it to exit prematurely if the git command or grep fails, and a suggestion was provided to ensure the fallback logic completes as intended.

Comment thread sdk-platform-java/.github/scripts/hermetic_library_generation.sh
@meltsufin meltsufin changed the base branch from main to release-please--fix-12825 April 18, 2026 02:43
@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

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

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant