Skip to content

test(firestore): kokoro to gcb - #8937

Merged
MarkDuckworth merged 45 commits into
mainfrom
markduckworth/kokoro-to-gcb-firestore
Jul 29, 2026
Merged

test(firestore): kokoro to gcb#8937
MarkDuckworth merged 45 commits into
mainfrom
markduckworth/kokoro-to-gcb-firestore

Conversation

@MarkDuckworth

@MarkDuckworth MarkDuckworth commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Fixes issues in #8700.

Because Datastore SDK tests also run against Firestore database instances (in Datastore mode) within long-door-651, naming collisions occur between the database instances used for Firestore and Datastore tests. To resolve this, Firestore CI will utilize uniquely named databases: firestore-standard and firestore-enterprise

We can merge this PR, or merge this branch to kokoro-to-gcb-firestore, and then merge #8700.

@MarkDuckworth
MarkDuckworth requested a review from a team as a code owner July 22, 2026 19:45
@product-auto-label product-auto-label Bot added the api: firestore Issues related to the Firestore API. label Jul 22, 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 adds a Cloud Build configuration file (cloudbuild.yaml) under handwritten/firestore to automate dependency installation, system test execution, and optional code coverage reporting. There are no review comments, and I have no feedback to provide.

@MarkDuckworth

Copy link
Copy Markdown
Contributor Author

/gcbrun

@MarkDuckworth

Copy link
Copy Markdown
Contributor Author

/gcbrun

@danieljbruce

Copy link
Copy Markdown
Contributor

This is looking good, but I would also suggest that we just skip the small number of tests that seem to be failing shown in https://paste.googleplex.com/4859917847625728. The GCB check will be a lot more reliable if we can expect it to pass under normal circumstances.

@MarkDuckworth

Copy link
Copy Markdown
Contributor Author

/gcbrun

@MarkDuckworth
MarkDuckworth enabled auto-merge (squash) July 29, 2026 15:22
@MarkDuckworth
MarkDuckworth merged commit 3fdcce4 into main Jul 29, 2026
36 of 37 checks passed
@MarkDuckworth
MarkDuckworth deleted the markduckworth/kokoro-to-gcb-firestore branch July 29, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the Firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants