Deprecate verbose option in logging

Option "verbose" from group "DEFAULT" is deprecated for removal.
The parameter has no effect.
Deprecated verbose for logging and init
Remove verbose in README
Remove verbose from tests.
If this option is not set explicitly, there is no such warning.

Change-Id: If60ca414d4f7cc7bd50aafb9a9ac8b328b5b5c84
This commit is contained in:
Iury Gregory Melo Ferreira 2016-05-17 23:58:14 -03:00
parent 881701c415
commit 05e25fdadc
6 changed files with 13 additions and 8 deletions

View File

@ -47,7 +47,6 @@ class { 'nova':
rabbit_password => 'an_even_bigger_secret', rabbit_password => 'an_even_bigger_secret',
image_service => 'nova.image.glance.GlanceImageService', image_service => 'nova.image.glance.GlanceImageService',
glance_api_servers => 'localhost:9292', glance_api_servers => 'localhost:9292',
verbose => false,
rabbit_host => '127.0.0.1', rabbit_host => '127.0.0.1',
} }
@ -75,12 +74,12 @@ nova is a combination of Puppet manifest and ruby code to delivery configuration
The `nova_config` provider is a children of the ini_setting provider. It allows one to write an entry in the `/etc/nova/nova.conf` file. The `nova_config` provider is a children of the ini_setting provider. It allows one to write an entry in the `/etc/nova/nova.conf` file.
```puppet ```puppet
nova_config { 'DEFAULT/verbose' : nova_config { 'DEFAULT/image_service' :
value => true, value => nova.image.glance.GlanceImageService,
} }
``` ```
This will write `verbose=true` in the `[DEFAULT]` section. This will write `image_service=nova.image.glance.GlanceImageService` in the `[DEFAULT]` section.
##### name ##### name

View File

@ -408,6 +408,10 @@ class nova(
warning('enabled_ssl_apis is empty but use_ssl is set to true') warning('enabled_ssl_apis is empty but use_ssl is set to true')
} }
if $verbose {
warning('verbose is deprecated, has no effect and will be removed after Newton cycle.')
}
if $use_ssl { if $use_ssl {
if !$cert_file { if !$cert_file {
fail('The cert_file parameter is required when use_ssl is set to true') fail('The cert_file parameter is required when use_ssl is set to true')

View File

@ -124,7 +124,7 @@ class nova::logging(
$debug_real = pick($::nova::debug,$debug) $debug_real = pick($::nova::debug,$debug)
if $verbose { if $verbose {
warning('verbose parameter is deprecated and has no effect. It\'s value will be ignored.') warning('verbose is deprecated, has no effect and will be removed after Newton cycle.')
} }
oslo::log { 'nova_config': oslo::log { 'nova_config':

View File

@ -0,0 +1,4 @@
---
deprecations:
- verbose option is now deprecated for removal, the
parameter has no effect.

View File

@ -35,7 +35,6 @@ describe 'basic nova' do
rabbit_password => 'an_even_bigger_secret', rabbit_password => 'an_even_bigger_secret',
image_service => 'nova.image.glance.GlanceImageService', image_service => 'nova.image.glance.GlanceImageService',
glance_api_servers => 'localhost:9292', glance_api_servers => 'localhost:9292',
verbose => true,
debug => true, debug => true,
rabbit_host => '127.0.0.1', rabbit_host => '127.0.0.1',
} }

View File

@ -78,8 +78,7 @@ describe 'nova' do
context 'with overridden parameters' do context 'with overridden parameters' do
let :params do let :params do
{ :verbose => true, { :debug => true,
:debug => true,
:log_dir => '/var/log/nova2', :log_dir => '/var/log/nova2',
:image_service => 'nova.image.local.LocalImageService', :image_service => 'nova.image.local.LocalImageService',
:rpc_backend => 'rabbit', :rpc_backend => 'rabbit',