Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

google cloud pubsub container do not expects keyword argument "channel" for publisher and subscriber client #161

Closed
Mohsin-Ul-Islam opened this issue Dec 2, 2021 · 1 comment
Closed

google cloud pubsub container do not expects keyword argument "channel" for publisher and subscriber client #161

Mohsin-Ul-Islam opened this issue Dec 2, 2021 · 1 comment
Labels
🔧 maintenance 📦 package: google 👀 requires attention

Comments

@Mohsin-Ul-Islam
Copy link
Contributor

Whenever I create an instance of PubsubContainer from google.cloud module, it raises an exception "Unexpected keyword argument 'channel' for PublisherClient.init".
@tillahoffmann tillahoffmann mentioned this issue Dec 13, 2021
Closed
@Mohsin-Ul-Islam
Copy link
Contributor Author

This issue came when I used google cloud pubsub client library version 2.5.0 via poetry. While the requirements in this project contain version 1.7.0.

Here is the release where channel keyword argument was passed to underlying pubsub clients https://googleapis.dev/python/pubsub/0.43.0/_modules/google/cloud/pubsub_v1/subscriber/client.html
But this is outdated and the implementation has been changed.

Instead now client_options and credentials are passed to the underlying pubsub clients. Here is the version reference I used
https://googleapis.dev/python/pubsub/2.5.0/_modules/google/cloud/pubsub_v1/publisher/client.html

In order to bump google-cloud-pubsub from 1.7.0 to latest I get the following error while running
pip-compile --verbose --upgrade-package google-cloud-pubsub==2.5.0

Could not find a version that matches docker[ssh]<4.3.0,>=5 (from -r requirements.in (line 7))
Tried: 2.0.0, 2.0.0, 2.0.1, 2.0.1, 2.0.2, 2.0.2, 2.1.0, 2.1.0, 2.2.0, 2.2.0, 2.2.1, 2.2.1, 2.3.0, 2.3.0, 2.4.0, 2.4.0, 2.4.1, 2.4.1, 2.4.2, 2.4.2, 2.5.0, 2.5.0, 2.5.1, 2.5.1, 2.6.0, 2.6.0, 2.6.1, 2.6.1, 2.7.0, 2.7.0, 3.0.0, 3.0.0, 3.0.1, 3.0.1, 3.1.0, 3.1.0, 3.1.1, 3.1.1, 3.1.2, 3.1.2, 3.1.3, 3.1.3, 3.1.4, 3.1.4, 3.2.0, 3.2.0, 3.2.1, 3.2.1, 3.3.0, 3.3.0, 3.4.0, 3.4.0, 3.4.1, 3.4.1, 3.5.0, 3.5.0, 3.5.1, 3.5.1, 3.6.0, 3.6.0, 3.7.0, 3.7.0, 3.7.1, 3.7.1, 3.7.2, 3.7.2, 3.7.3, 3.7.3, 4.0.0, 4.0.0, 4.0.1, 4.0.1, 4.0.2, 4.0.2, 4.1.0, 4.1.0, 4.2.0, 4.2.0, 4.2.1, 4.2.1, 4.2.2, 4.2.2, 4.3.0, 4.3.0, 4.3.1, 4.3.1, 4.4.0, 4.4.0, 4.4.1, 4.4.1, 4.4.2, 4.4.2, 4.4.3, 4.4.3, 4.4.4, 4.4.4, 5.0.0, 5.0.0, 5.0.1, 5.0.1, 5.0.2, 5.0.2, 5.0.3, 5.0.3
There are incompatible versions in the resolved dependencies:
docker<4.3.0 (from -r requirements.in (line 7))
docker (from testcontainers===dev->-r requirements.in (line 1))
docker[ssh]>=5 (from docker-compose==1.29.2->testcontainers===dev->-r requirements.in (line 1))

@tillahoffmann tillahoffmann mentioned this issue Dec 15, 2021
Merged
alexanderankin added a commit that referenced this issue Mar 30, 2024
@Mohsin-Ul-Islam @alexanderankin
fix: pass updated keyword args to Publisher/Subscriber client in goog…
8addc11 
…le/pubsub #161 (#164)

…le/pubsub

---------

Co-authored-by: David Ankin <daveankin@gmail.com>
@github-actions github-actions bot mentioned this issue Mar 30, 2024
Merged
alexanderankin pushed a commit that referenced this issue Apr 1, 2024
@github-actions
chore(main): release testcontainers 4.3.0 (#500)
f819c7a 
🤖 I have created a release *beep* *boop*
---


##
[4.3.0](testcontainers-v4.2.0...testcontainers-v4.3.0)
(2024-04-01)


### Features

* **client:** Add custom User-Agent in Docker client as
`tc-python/&lt;version&gt;`
([#507](#507))
([dd55082](dd55082))


### Bug Fixes

* Add CassandraContainer
([#476](#476))
([507e466](507e466))
* add chroma container
([#515](#515))
([0729bf4](0729bf4))
* Add Weaviate module
([#492](#492))
([90762e8](90762e8))
* **cassandra:** make cassandra dependency optional/test-only
([#518](#518))
([bddbaeb](bddbaeb))
* **core:** allow setting docker command path for docker compose
([#512](#512))
([63fcd52](63fcd52))
* **google:** add support for Datastore emulator
([#508](#508))
([3d891a5](3d891a5))
* Improved Oracle DB module
([#363](#363))
([6e6d8e3](6e6d8e3))
* inconsistent test runs for community modules
([#497](#497))
([914f1e5](914f1e5))
* **kafka:** Add redpanda testcontainer module
([#441](#441))
([451d278](451d278))
* **kafka:** wait_for_logs in kafka container to reduce lib requirement
([#377](#377))
([909107b](909107b))
* **keycloak:** container should use dedicated API endpoints to
determine container readiness
([#490](#490))
([2e27225](2e27225))
* **nats:** Client-Free(ish) NATS container
([#462](#462))
([302c73d](302c73d))
* **new:** add a new Docker Registry test container
([#389](#389))
([0f554fb](0f554fb))
* pass doctests, s/doctest/doctests/, run them in gha,
s/asyncpg/psycopg/ in doctest, fix keycloak flakiness: wait for first
user
([#505](#505))
([545240d](545240d))
* pass updated keyword args to Publisher/Subscriber client in
google/pubsub
[#161](#161)
([#164](#164))
([8addc11](8addc11))
* Qdrant module
([#463](#463))
([e8876f4](e8876f4))
* remove accidentally added pip in dev dependencies
([#516](#516))
([dee20a7](dee20a7))
* **ryuk:** Enable Ryuk test suite. Ryuk image 0.5.1 -&gt; 0.7.0. Add
RYUK_RECONNECTION_TIMEOUT env variable
([#509](#509))
([472b2c2](472b2c2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🔧 maintenance 📦 package: google 👀 requires attention
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tillahoffmann @Mohsin-Ul-Islam