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

Fix compilation failures on RHEL when manage_package => true #431

Merged
merged 5 commits into from
Dec 21, 2017
Merged

Fix compilation failures on RHEL when manage_package => true #431

merged 5 commits into from
Dec 21, 2017

Conversation

fatmcgav
Copy link
Contributor

@fatmcgav fatmcgav commented Dec 21, 2017

Fixes #430

TODO:

  • Improve testing
  • Implement fix for issue

create_admin: true,
admin_username: 'admin',
admin_password: 'password'
on_supported_os.each do |os, facts|
Copy link
Member

Choose a reason for hiding this comment

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

yey rspec puppet facts \o/

@@ -1,177 +1,199 @@
require 'spec_helper'

describe 'mongodb::server' do
let :facts do
# Default facts
let(:default_facts) do
Copy link
Member

Choose a reason for hiding this comment

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

this block shouldnt be needed. We already mock this fact here: https://github.com/voxpupuli/puppet-mongodb/blob/master/spec/default_module_facts.yml

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, that's a useful method to know, as we have a number of occurrences of this on internal module tests...

@fatmcgav
Copy link
Contributor Author

And we're green!

@bastelfreak Did you want me to squash the commits down?
Or happy with this PR in it's current form?

@fatmcgav fatmcgav changed the title [WIP] Fix compilation failures on RHEL when manage_package => true Fix compilation failures on RHEL when manage_package => true Dec 21, 2017
@bastelfreak
Copy link
Member

Thanks for the PR!

@bastelfreak bastelfreak merged commit 990ec6f into voxpupuli:master Dec 21, 2017
@bastelfreak bastelfreak added the bug Something isn't working label Dec 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Unknown variable 'mongodb::params::journal'" error when 'manage_package => true'
2 participants