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

Add broker_state option to rabbitmq_plugin. #35

Merged

Conversation

Shadow6363
Copy link
Contributor

@Shadow6363 Shadow6363 commented Jun 29, 2020

SUMMARY

Adds a broker_state option to rabbitmq_plugin that controls whether or not an attempt is made to connect to the node to change the plugin state.
Some plugins require the plugin enabled state to be modified directly, without connecting to the node first.
An example of this is AWS Peer Discovery Plugin.

I wasn't sure what to call the option so if you have better suggestions, please let me know.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

rabbitmq_plugin

ADDITIONAL INFORMATION

Further information on the option can be found in the official RabbitMQ documentation on the command.

@Shadow6363
Copy link
Contributor Author

I think the only tests that failed are from network problems, but I don't have permission to retry tests.

Copy link
Collaborator

@odyssey4me odyssey4me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll look into the test issue and try to get that resolved ASAP.

plugins/modules/rabbitmq_plugin.py Show resolved Hide resolved
@Shadow6363
Copy link
Contributor Author

All tests worked and passed this time. Thanks again for the prompt review!

@odyssey4me odyssey4me merged commit da64420 into ansible-collections:master Jul 1, 2020
ChandlerSwift pushed a commit to ChandlerSwift/community.rabbitmq that referenced this pull request Dec 22, 2020
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.

2 participants