This module still supports Puppet 5. This change adds back unit test
coverage with Puppet 5.
Depends-on: https://review.opendev.org/854061
Change-Id: I757d5c2f1c2af1c548d8ce6fba324c6f42826910
Capitalized values of True and False are conflicting with systemd
scripts
This commit set all values to lowercase
Fixes: Bug #1697476
Change-Id: Ibd01a7a850e7593fabd676b8b0c0af2b77101a2a
spec_helpers didn't change because keystone module (>=5.1.0 <6.0.0)
is not present in openstack/keystone - only >= 6.0.0 and it will
not working for puppet-ceph acceptance tests, because they are running
with Juno code.
Change-Id: Ibf7c55cac755432fb4901587693363cea4c27f42
This patch aim to update our specs test in order to work with the new
rspec-puppet release 2.0.0, in the mean time, we update rspec syntax
in order to be prepared for rspec 3.x move.
In details:
* Convert 'should' keyword to 'is_expected.to' (prepare rspec 3.x)
* Upgrade and pin rspec-puppet from 1.0.1 to 2.0.0
* Clean Gemfile (remove over-specificication of runtime deps of puppetlabs_spec_helper)
Change-Id: I512ffe4ac12763ec5a4bea935001088bad231cc1
In the recent release 1.1.1 of puppet inifile type the exists? call
has been changed to return booleans instead of nil / object
This changes the dependant unit test to mirror this change.
Change-Id: I768948250b3269dffa766d05ddd03d2ea20bcdd3
Closes-Bug: #1340319
* add a ini_file helper as ceph_config.
* set the seperator as ' = ' in order to be the same as ceph-deploy
* add path param so that seperator can be easily tested.
** also allows for users to change the default path if desired.
* add test case to prove spacing around seperator.
* add simple test case to ensure type works.
Closes-bug: #1243852
Change-Id: I0f3bb41d75d48d8af3b5b7cfb7768c4d3de2c002