Switch Nova to use $::os_service_default
Change logging.pp, db.pp and tests.
Change-Id: I928a93534c6d27c020b7afb5b7dda32c379e9d62
Related-bug: #1515273
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
order to be prepared for rspec 3.x move.
In details:
* Upgrade and pin rspec-puppet from 1.0.1 to 2.0.0
* 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
* Clean Gemfile (remove over-specificication of runtime deps of puppetlabs_spec_helper)
Change-Id: I172439c6ed185bb38b325b2524cab1475cdc7504
Change default charset to utf8 and default collation order to
utf8_unicode_ci to align with upstream defaults.
Change-Id: I2aa59fd868476f599019e3029af36aa707139fe1
Closes-Bug: #1302088
In the origin nova::db::mysql, if the value of $allowed_hosts
contains or equals to $host, then puppet will complain duplicate
declaration error. This patch is aim to update the allowed_hosts
conditonal statement in nova::db::mysql.
There are two cases to pass $allowed_hosts to $real_allowed_hosts:
- If $allowed_hosts is array,then remove $host from $allowed_hosts;
- elsif $allowed_hosts is string and not equivalent to $host;
At last, if $real_allowed_hosts is not undef, then run
nova::db::mysql::host_access
Fix bug 1206444
Change-Id: If018321095e62e1196e0c2e6b623b30acb535020
This massive code commit actually implements something very simple.
previously, we allowed nova_config to omit a section and assumed that
section was default.
This commit updates the code to require section names for all settings.
This change is being made b/c:
- it better maps to the config on disk
- it is consistent with the other modules
Change-Id: Iae71a4c48ed0f9792566f16f0bf13e61569b46e5
Allow to specify the database charset but still default to 'latin1' if
none is provided.
Remove nova::params::nova_db_charset. Both osfamily were using 'latin1'
since the update for Folsom in 76d4632.
