googleapis / python-bigtable

Apache License 2.0
64 stars 55 forks source link

Unable to use patched version of grpcio with google-cloud-bigtable #981

Open cglewis opened 3 weeks ago

cglewis commented 3 weeks ago

There are a number of bugs (functionally and security wise) in grpcio < 1.63. However, >1.63 now requires >5 protobuf, but this library requires protobuf <5 making it no longer possible to install this library with a patched working version of grpcio.

Because no versions of google-cloud-bigtable match >2.23.1,<3.0.0
 and google-cloud-bigtable (2.23.1) depends on protobuf (>=3.19.5,<3.20.0 || >3.20.0,<3.20.1 || >3.20.1,<4.21.0 || >4.21.0,<4.21.1 || >4.21.1,<4.21.2 || >4.21.2,<4.21.3 || >4.21.3,<4.21.4 || >4.21.4,<4.21.5 || >4.21.5,<5.0.0dev), google-cloud-bigtable (>=2.23.1,<3.0.0) requires protobuf (>=3.19.5,<3.20.0 || >3.20.0,<3.20.1 || >3.20.1,<4.21.0 || >4.21.0,<4.21.1 || >4.21.1,<4.21.2 || >4.21.2,<4.21.3 || >4.21.3,<4.21.4 || >4.21.4,<4.21.5 || >4.21.5,<5.0.0dev).
Because grpcio (1.64.1) depends on protobuf (>=5.26.1,<6.0dev)
 and no versions of grpcio match >1.64.1,<2.0.0, grpcio (>=1.64.1,<2.0.0) requires protobuf (>=5.26.1,<6.0dev).
Thus, google-cloud-bigtable (>=2.23.1,<3.0.0) is incompatible with grpcio-reflection (>=1.64.1,<2.0.0).

Can support for a newer version of protobuf be included in this library?

cglewis commented 2 weeks ago

@mukund-ananthu thanks for getting this fixed in github.com/googleapis/python-pubsub any chance you can help getting it fixed here as well?

mukund-ananthu commented 2 weeks ago

@cglewis Unfortunately, I do not handle this library. You could ping / check with the assignee and / or raise a customer ticket for faster resolution.

cglewis commented 2 weeks ago

@rkaregar can you take a look please, when you have a moment? this is blocking usage of this library for us. it seemed like a fairly easy fix for the pubsub library: https://github.com/googleapis/python-pubsub/pull/1193