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.

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

Unit test test_nack_calls_ack test hangs with google-cloud-pubsub==2.10.0 #311

Closed
parthea opened this issue Mar 6, 2022 · 0 comments · Fixed by #312 or #313
Closed

Unit test test_nack_calls_ack test hangs with google-cloud-pubsub==2.10.0 #311

parthea opened this issue Mar 6, 2022 · 0 comments · Fixed by #312 or #313
Assignees
Labels
api: pubsublite Issues related to the googleapis/python-pubsublite API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@parthea
Copy link
Contributor

parthea commented Mar 6, 2022

async def test_nack_calls_ack(
subscriber: SinglePartitionSingleSubscriber,
underlying,

Unit test test_nack_calls_ack() started timing out in main after google-cloud-pubsub==2.10.0 was released. See build log here and the checks in PR #310.

Test test_nack_calls_ack() no longer hangs when I change the google-cloud-pubsub dependency in setup.py from "google-cloud-pubsub >= 2.1.0, <3.0.0dev", to "google-cloud-pubsub >= 2.1.0, <3.0.0dev, !=2.10.0",

@parthea parthea added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Mar 6, 2022
@product-auto-label product-auto-label bot added the api: pubsublite Issues related to the googleapis/python-pubsublite API. label Mar 6, 2022
dpcollins-google added a commit that referenced this issue Mar 7, 2022
dpcollins-google added a commit that referenced this issue Mar 7, 2022
dpcollins-google added a commit that referenced this issue Mar 7, 2022
dpcollins-google added a commit that referenced this issue Mar 7, 2022
dpcollins-google added a commit that referenced this issue Mar 7, 2022
dpcollins-google added a commit that referenced this issue Mar 7, 2022
…om the cloud Pub/Sub Message (#312)

* fix: ignore the implementation details of message acknowledgements from the cloud Pub/Sub Message

fixes: #311

* fix: ignore the implementation details of message acknowledgements from the cloud Pub/Sub Message

fixes: #311

* fix: ignore the implementation details of message acknowledgements from the cloud Pub/Sub Message

fixes: #311

* fix: ignore the implementation details of message acknowledgements from the cloud Pub/Sub Message

fixes: #311

* fix: ignore the implementation details of message acknowledgements from the cloud Pub/Sub Message

fixes: #311
gcf-merge-on-green bot pushed a commit that referenced this issue Mar 7, 2022
🤖 I have created a release *beep* *boop*
---


### [1.4.1](v1.4.0...v1.4.1) (2022-03-07)


### Bug Fixes

* ignore the implementation details of message acknowledgements from the cloud Pub/Sub Message ([#312](#312)) ([19da693](19da693)), closes [#311](#311)

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsublite Issues related to the googleapis/python-pubsublite API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
2 participants