Don't define virsh secret if already defined

We test to see if the ceph secret already has been defined before
installing the secret again.

This is mainly done to keep our Foreman runs clean, as it reports
on hosts that have had changes applied. Without the 'unless' clause
any node that has the nova compute class installed will report as
having changes applied.

Change-Id: I05e83f07570d4a0e67a700bd9d9030eef1c261c3
Update quota flags

Sync puppet module flags with latest nova quota parameters.

Changes:
 - Flags 'quota_volumes' and 'quota_gigabytes' is now deprecated and has no effect
 - Add 'quota_injected_files', 'quota_injected_file_content_bytes', 'quota_injected_file_path_bytes'
 - Flags 'quota_max_injected_*' is now deprecated and has effect for the same 'quota_injected_*'
 - Add 'quota_fixed_ips' and 'until_refresh'

Reference:
 - Standardize quota flag format
   https://github.com/openstack/nova/commit/97074667b58a980de0a2ddd2d1ddfc6078ecfc75

Change-Id: Ia67e80725ed2b9171950764729143f7307fef450
Deprecate conductor_workers in nova::api

Since we now have a dedicated nova::conductor class, the
conductor_workers parameter in nova::api isn't at his right place.
This commit deprecate the conductor_workers parameter in profit of
workers param in nova::conductor ( see https://review.openstack.org/106656/ )

Change-Id: I767b5c143f12a0f518828287b33aedbfd2201e72
Ensure conductor_workers stay at previous value

During the refactoring of the conductor_workers parameter from nova::api
to nova::conductor, the conductor_workers default changed from
$::processorcount to the parameter being absent.

While this is consistent with the nova default behaviour, it changes the
value for people who currently have a different default set. The solution is
to do nothing, in case no value is given.

For reference see:
https://review.openstack.org/#/c/106656
https://review.openstack.org/#/c/108941
http://docs.openstack.org/trunk/config-reference/content/list-of-compute-config-options.html

Change-Id: I908fc4953522f0b1db8f43882a44407aba9fa9f9
Merge "Deprecate conductor_workers in nova::api"
Merge "Ensure conductor_workers stay at previous value"
Remove dynamic scoping with File from class nova.

Resource defaults are applied to all file resources
in class nova using File. In some circumstances the
scope of these defaults becomes too wide, causing
a dependency cycle. This commit removes the dynamic
scoping and instead explicitly defines the parameters
of each file resource to avoid creating dependency
cycles. Functionality should otherwise be identical.

Change-Id: I41d85e65ea73c54ce1a8be292ef3f781e17e48a7
Closes-Bug: 1349765
Merge "Update quota flags"
Merge "Remove dynamic scoping with File from class nova."
Merge "Don't define virsh secret if already defined"
Merge git://github.com/puppetlabs/puppetlabs-nova