Skip to content

Do not log every transient (or Erlang/OTP-provided) dependency when listing plugins #14373

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

Merged
merged 1 commit into from
Aug 12, 2025

Conversation

michaelklishin
Copy link
Collaborator

@michaelklishin michaelklishin commented Aug 12, 2025

The code path in question is executed every time
rabbit_plugins:list/2 (e.g. rabbit_plugins:is_enabled/1) is used, which with some distributed plugins can
happen once or several times a minute.

Given the maturity of the plugins subsystem, we
arguably can drop those messages.

The code path in question is executed every time
rabbit_plugins:list/2 (e.g. rabbit_plugins:is_enabled/1)
is used, which with some distributed plugins can
happen once or several times a minute.

Given the maturity of the plugins subsystem, we
arguably can drop those messages.
@michaelklishin michaelklishin added this to the 4.2.0 milestone Aug 12, 2025
@michaelklishin michaelklishin changed the title Do not log every transient dependency when listing plugins Do not log every transient (or Erlang/OTP-provided) dependency when listing plugins Aug 12, 2025
@michaelklishin michaelklishin merged commit 64e5a6a into main Aug 12, 2025
560 of 561 checks passed
@michaelklishin michaelklishin deleted the mk-reduce-rabbitmq-plugins-log-rate branch August 12, 2025 19:00
michaelklishin added a commit that referenced this pull request Aug 12, 2025
michaelklishin added a commit that referenced this pull request Aug 12, 2025
Do not log every transient (or Erlang/OTP-provided) dependency when listing plugins (backport #14373)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant