Skip to content

fix(spanner): system test against regular sessions#8872

Merged
alkatrivedi merged 1 commit into
mainfrom
fix-system-test
Jul 14, 2026
Merged

fix(spanner): system test against regular sessions#8872
alkatrivedi merged 1 commit into
mainfrom
fix-system-test

Conversation

@alkatrivedi

@alkatrivedi alkatrivedi commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

This PR is to fix system test against regular sessions which are failing due to running them in Cloud Builds.

@alkatrivedi alkatrivedi requested a review from a team as a code owner July 13, 2026 20:10
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the Spanner API. label Jul 13, 2026

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

Copy link
Copy Markdown
Contributor

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 simplifies the error assertions in the Spanner system tests by removing conditional checks on KOKORO_JOB_NAME and directly asserting that the error code is grpc.status.INVALID_ARGUMENT. I have no feedback to provide as there are no review comments to evaluate.

@alkatrivedi alkatrivedi force-pushed the fix-system-test branch 7 times, most recently from 1dec43a to a673c48 Compare July 14, 2026 08:24
@alkatrivedi alkatrivedi merged commit bd23de1 into main Jul 14, 2026
35 checks passed
@alkatrivedi alkatrivedi deleted the fix-system-test branch July 14, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the Spanner API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants