-
-
Notifications
You must be signed in to change notification settings - Fork 497
add FreeBSD support #542
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 FreeBSD support #542
Conversation
| should contain_package('rabbitmq-server').with( | ||
| 'ensure' => 'installed', | ||
| 'name' => 'rabbitmq', | ||
| 'provider' => 'freebsd' |
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.
This is why your test is failing, it's expecting pkgng.
Though I think provider isn't actually needed. Not sure if #493 will get merged, but would have to adjust slightly to get FreeBSD support working if / when that gets merged.
| $package_name = 'rabbitmq' | ||
| $service_name = 'rabbitmq' | ||
| $package_provider = 'pkgng' | ||
| $version = '3.4.2' |
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.
Also, keep in mind that while $version is used internally as things work now, it's ignored when using the system package provider, so it doesn't really matter in this case.
|
ps - https://github.com/voxpupuli/puppet-rabbitmq/blob/master/metadata.json may also need to be updated (FreeBSD and OpenBSD support are both not currently included there) |
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.
metadata.json probably needs updating, and test needs to be fixed
|
@olevole would you like to rework this one? If not, I can give a quick go at reworking it and submitting a new PR. |
|
@wyardley Hi Will! |
|
I can take a stab at it once #493 is merged, shouldn't be hard. |
|
@olevole ps - one question, though - is |
|
Haven't submitted yet, but check out this: |
|
reworking via #567 |
No description provided.