Skip to content

tests: remove protobuf cpp from tests which is no longer used as of Protobuf 4.x#17742

Open
parthea wants to merge 3 commits into
mainfrom
remove-cpp-spanner
Open

tests: remove protobuf cpp from tests which is no longer used as of Protobuf 4.x#17742
parthea wants to merge 3 commits into
mainfrom
remove-cpp-spanner

Conversation

@parthea

@parthea parthea commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Both google-cloud-spanner and google-cloud-bigtable only support Protobuf 6.x+ . The tests for protobuf implementation backend cpp are no longer needed .

"protobuf >= 6.33.5, < 8.0.0",

"protobuf >= 6.33.5, < 8.0.0",

This PR also skips Python 3.15 testing for google-cloud-bigtable because it depends on 3.15 wheels from google-crc32c. The follow up work is tracked in #17741

Note: Python 3.15 tests are not enabled yet. That will be part of #17585

@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 updates the test configuration for google-cloud-bigtable and google-cloud-spanner by adding support for Python 3.15 and removing the deprecated 'cpp' protobuf implementation from all test sessions. Additionally, it includes a temporary skip for Python 3.15 unit tests pending the availability of google-crc32c wheels. I have no feedback to provide.

@parthea
parthea marked this pull request as ready for review July 17, 2026 17:55
@parthea
parthea requested review from a team as code owners July 17, 2026 17:55
@hebaalazzeh
hebaalazzeh self-requested a review July 17, 2026 17:57

@hebaalazzeh hebaalazzeh 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.

LGTM

@parthea
parthea enabled auto-merge (squash) July 17, 2026 18:09
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.

2 participants