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
rabbitmq-plugins list: allow warnings to be suppressed #10865
Conversation
to optionally allow avoiding warning banner
deps/rabbitmq_cli/lib/rabbitmq/cli/plugins/commands/list_command.ex
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have only now noticed that the example uses --formatter=json
.
For such cases, there are already --silent
, --quiet
and RabbitMQ.CLI.Core.Config.output_less?/1
.
Instead of a new option, we can simply use RabbitMQ.CLI.Core.Config.output_less?/1
where ignore_warnings
is used in this PR to ignore the warnings.
See #10870. |
(cherry picked from commit c0187ec)
nice, reusing @michaelklishin Thank you! |
Proposed Changes
Some tools list plugins e.g. in
json
format, and can fail to decode output from missing plugins warning banner. We want to be able to optionally list plugins with no warning banner - avoiding any extra work on calling tools (e.g. in the case of sharing config files across new and legacy brokers). Example, withrabbitmq_unknown
in enabled_plugins file:with warning
with no warning
NOTE: --silent and --quiet options more for generic CommandBehaviour messages
Types of Changes
What types of changes does your code introduce to this project?
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply.You can also fill these out after creating the PR.
If you're unsure about any of them, don't hesitate to ask on the mailing list.
We're here to help!
This is simply a reminder of what we are going to look for before merging your code.
CONTRIBUTING.md
documentFurther Comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc.