openstack/kayobe
Code Issues Proposed changes

Respect comments about inspector_keep_ports

Browse Source 
The previous behaviour did not match the comments.

Change-Id: Ifc7bb3e34a0d8aaa6a6c2dc534f3b9603b1a76ab
This commit is contained in:
Will Szumski 2022-02-17 16:22:25 +00:00
parent 6e404d1fe4
commit a1409994e4
2 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ansible/roles/kolla-openstack/templates/kolla/config/ironic-inspector.conf
View File

@ -13,8 +13,7 @@ add_ports = {{ kolla_inspector_add_ports }}
{% endif %} {% endif %}
{% if kolla_inspector_keep_ports %} {% if kolla_inspector_keep_ports %}
# Keep only those ports that were found during inspection. keep_ports = {{ kolla_inspector_keep_ports }}
keep_ports = present
{% endif %} {% endif %}
# Store logs returned by the inspection ramdisk. # Store logs returned by the inspection ramdisk.

11
releasenotes/notes/respect-comment-for-keep-ports-e0b1ab73395055a6.yaml Normal file
View File

@ -0,0 +1,11 @@
---
upgrade:
- |
If you have customized ``inspector_keep_ports``, ensure it is set to one of:
``all``, ``present``, ``added``.
fixes:
- |
``inspector_keep_ports`` can be set to one of: ``all``, ``present``,
``added``. The previous behavior was to set this to ``present`` if the
variable was changed to any non-None value. This respects the behavior
indicated in the comments.