Support for defining operator policies?

[philomory]

Is there a means of defining operator policies in this module? Obviously there's not a rabbitmq_operator_policy type, but, I wasn't sure if there was some other means of defining them, such as supplying a particular argument to the rabbitmq_policy type (I couldn't see one when inspecting the source code, but I could have just missed it).

If not, then given the existence of the rabbitmqctl set_operator_policy command, it seems like a rabbitmq_operator_policy type could be implemented by basically just copying the rabbitmq_policy type and switching out the rabbitmqctl command of set_policy for set_operator_policy (since the two commands take all the same set of arguments and flags).

[wyardley]

I'm not too sure - I'm not up on the latest and greatest in RabbitMQ, but I would assume if it's a relatively newer feature, it's probably not supported in the module.

From a very quick look at the docs and the commands used to define operator policies, I believe that in theory, what you say is correct - new types / providers / unit tests could be added that would likely be pretty similar to rabbitmq_policy, with some caveats.

• It might be hard to test or add integration tests for this until something like BREAKING: Enable packagecloud repos by default #926 ships, because the version of RabbitMQ in the default settings is relatively old. This might also make development / testing of the module a bit slower locally (because of this, you'd want to make sure to do extra functional testing to make sure everything works well under a range of conditions). That said, I'm open to approving a PR to add this support without that assuming there are good docs / unit tests, and all the CI checks pass, and you've done some functional testing.
• Probably it would need some specific guards to make sure it will only work with supported RMQ versions
• You'd have to account for differences in any other features that are different from rabbitmq_policy. I know you said the flags are basically the same, so that may not be a huge issue.