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

Revert "Remove dead_queue_rejects test flake" #2325

Merged
merged 1 commit into from
Apr 27, 2020

Conversation

lukebakken
Copy link
Collaborator

@lukebakken lukebakken commented Apr 22, 2020

This reverts commit 9772aa8

Changing from amqp_channel:call to amqp_channel:cast allows the operation to complete before a new queue process is started, resulting in the expected basic.nack response.

@lukebakken lukebakken self-assigned this Apr 22, 2020
This reverts commit 9772aa8.

Check RPC return value

Try a cast to fix test
@lukebakken lukebakken force-pushed the revert-rabbitmq-server-2308-dead_queue_rejects branch from c8b9c2b to 53cc7b4 Compare April 23, 2020 19:59
@lukebakken lukebakken marked this pull request as ready for review April 24, 2020 13:33
@lukebakken lukebakken requested review from gerhard and dumbbell April 24, 2020 13:33
@lukebakken lukebakken changed the title Revert "Remove dead_queue_rejects test flake" Revert removing some flaky tests Apr 27, 2020
@lukebakken lukebakken marked this pull request as draft April 27, 2020 14:52
@lukebakken lukebakken changed the title Revert removing some flaky tests Revert "Remove dead_queue_rejects test flake" Apr 27, 2020
@lukebakken lukebakken marked this pull request as ready for review April 27, 2020 14:54
@dumbbell dumbbell merged commit 55d28f5 into master Apr 27, 2020
@dumbbell dumbbell deleted the revert-rabbitmq-server-2308-dead_queue_rejects branch April 27, 2020 15:04
lukebakken pushed a commit that referenced this pull request Apr 27, 2020
…ad_queue_rejects

Revert "Remove dead_queue_rejects test flake"

(cherry picked from commit 55d28f5)
@michaelklishin
Copy link
Member

👍

lukebakken added a commit that referenced this pull request Apr 30, 2020
…-2308-dead_queue_rejects"

This reverts commit 55d28f5, reversing
changes made to 0bf8d08.
lukebakken added a commit that referenced this pull request Apr 30, 2020
…-2308-dead_queue_rejects"

This reverts commit 9b80b9c.
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.

3 participants