In order to be able to take an action after all the packages of the
module have been installed/updated or all the services have been
started/restarted, we set a 'cinder-package' and 'cinder-service' tag
for each package and service of this module.
At the moment, there is a generic openstack tag that is not specific
enough if one wants to take action upon a single module change.
Use case :
If an action needs to be taken after all the packages have been
installed or updated : Package <| tag == 'cinder-package' |> -> X
Change-Id: Ifa40d902263afa019395e1f12c3063ff3a11bff0
Some users wish to override the default package provider by their own.
Tag all packages with the 'openstack' to allow mass resource attributes
override using resource collectors.
Closes-bug: #1391209
Change-Id: I5e52716c378ee80e7071c459d9deccb103cb18ad
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
This patch aim to update our specs test in order to work with the
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:
* Use shared_examples "a Puppet::Error" for puppet::error tests
* Convert 'should' keyword to 'is_expected.to' (prepare rspec 3.x)
* Fix spec tests for rspec-puppet 2.0.0
* 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: Id5b428fb518f40cf92cd27078d36f19b6d60226b
Card: https://trello.com/c/eHXc1Ryd/4-investigate-the-necessary-change-to-be-rspec-puppet-2-0-0-compliant
Per rspec-puppet-1.0.1 warning:
DEPRECATION: include_class is deprecated. Use contain_class instead.
Called from .../vendor/gems/rspec-puppet-1.0.1/lib/rspec-puppet/
matchers/include_class.rb:7
Change-Id: I6b410bc4d294bdc6f4feb65a18b213f88b5184f1
In my last commit on Cinder module, I add cinder::backup::swift
without adding cinder::backup class which is useful to ensure that
package is installed and service is running.
It also includes variables related to backup configuration.
Note: swift is one of the drivers of backup in Cinder, other ones should
be included in manifests/backup/<driver>.pp
Change-Id: I1a55245abe61f9bbd71568d76ed00e0ecf08307e
Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>
