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

rabbit_channel: Ignore DOWN message from monitored process if it exited normally #13336

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

dumbbell
Copy link
Member

Why

It happens in CI from time to time and it was crashing the channel process. There is always a channel.close method pending in the channel mailbox.

How

For now, log something and ignore the DOWN message. The channel will exit after handling the pending channel.close method anyway.

@dumbbell dumbbell self-assigned this Feb 13, 2025
…ed normally

[Why]
It happens in CI from time to time and it was crashing the channel
process. There is always a `channel.close` method pending in the
channel mailbox.

[How]
For now, log something and ignore the DOWN message. The channel will
exit after handling the pending `channel.close` method anyway.
@dumbbell dumbbell force-pushed the ignore-normal-exit-in-channel branch from 7bee3c4 to 8945b75 Compare March 12, 2025 08:33
@dumbbell dumbbell marked this pull request as ready for review March 12, 2025 16:44
@dumbbell dumbbell merged commit 7c2ee95 into main Mar 12, 2025
274 checks passed
@dumbbell dumbbell deleted the ignore-normal-exit-in-channel branch March 12, 2025 16:44
@michaelklishin
Copy link
Member

@Mergifyio backport v4.1.x v4.0.x

Copy link

mergify bot commented Mar 12, 2025

backport v4.1.x v4.0.x

✅ Backports have been created

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.

None yet

4 participants