openstack/ironic
Code Issues Proposed changes

Fix default value of "ignore_req_list" config option

Browse Source 
If audit is enabled ironic API service does not start with default
value (None) of "ignore_req_list" config option. This patch changes
default value to empty string. Related modification added to ironic
sample config.

Closes-Bug: 1648387
Change-Id: Ifbbbed4012670b4b05fcf63e5586024f3c927875
This commit is contained in:
Yuriy Zveryanskyy 2016-12-08 13:19:00 +02:00
parent b15a8cf90a
commit 8bcee6e206
3 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
etc/ironic/ironic.conf.sample
View File

@ -663,7 +663,7 @@
# be done on any GET or POST requests if this is set to # be done on any GET or POST requests if this is set to
# "GET,POST". It is used only when API audit is enabled. # "GET,POST". It is used only when API audit is enabled.
# (string value) # (string value)
#ignore_req_list = <None> #ignore_req_list =
[cimc] [cimc]

1
ironic/conf/audit.py
View File

@ -26,6 +26,7 @@ opts = [
'Used only when API audit is enabled.')), 'Used only when API audit is enabled.')),
cfg.StrOpt('ignore_req_list', cfg.StrOpt('ignore_req_list',
default='',
help=_('Comma separated list of Ironic REST API HTTP methods ' help=_('Comma separated list of Ironic REST API HTTP methods '
'to be ignored during audit logging. For example: ' 'to be ignored during audit logging. For example: '
'auditing will not be done on any GET or POST ' 'auditing will not be done on any GET or POST '

4
releasenotes/notes/bug-1648387-92db52cbe007fabd.yaml Normal file
View File

@ -0,0 +1,4 @@
---
fixes:
- Fixes the issue that API service does not start if audit is enabled with
default value of "ignore_req_list" config option.