voxpupuli / puppet-rabbitmq

RabbitMQ Puppet Module
http://forge.puppetlabs.com/puppet/rabbitmq
Apache License 2.0
171 stars 500 forks source link

Treat `initial-cluster-size` option in policy as an integer #950

Closed jimmybigcommerce closed 1 year ago

jimmybigcommerce commented 1 year ago

Pull Request (PR) description

Similiar to https://github.com/voxpupuli/puppet-rabbitmq/pull/628, the policy setting initial-cluster-size needs to be parsed into an integer before being passed to RMQ.

Fixes this:

Error:
Validation failed

<<"3">> is not a valid cluster size
jimmybigcommerce commented 1 year ago

LGTM If it's not too much trouble, would you mind squashing all the commits one more time after that last suggestion?

I just did a release today, but will try to cut a release for this one fairly soon. If you want to take a stab at making the release PR, it will help:

https://voxpupuli.org/docs/releasing_version/ @wyardley Have squashed. I don't have perms to merge. If you merge it, I'll have a stab at the release