-
-
Notifications
You must be signed in to change notification settings - Fork 162
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 SLES 12/15 Support #462
Conversation
Thus far, there is no SLES added to the acceptance tests, which might be of interest. |
bc539f0
to
0db450b
Compare
spec/classes/corosync_spec.rb
Outdated
context 'on SLES platforms' do | ||
let :facts do | ||
{ | ||
osfamily: 'Suse', |
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.
you also need to mock the os
hash here. I'm not sure if it's easier or harder to migrate the whole file to rspec-puppet-facts 🤔
Dear @towo, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
This needs a rebase now. |
0db450b
to
682c896
Compare
7acfd86
to
cdf8f38
Compare
Going by corosync/pacemaker versions in SLES 12 and 15, I've added sane-ish defaults. Bonus points for checking for existence of SLE-HA repositories; checking if we have a registered HA product should be an optional feature.
cdf8f38
to
961637f
Compare
Pull Request (PR) description
Adds support for SUSE Linux Enterprise Server to the puppetlabs-corosync module.
No special care is given to ensure only supported clusters are used, that's for either a profile or a separate module.
This Pull Request (PR) fixes the following issues
n/a