openstack/ironic
Code Issues Proposed changes

redfish: change default virtual media storage to local storage

Browse Source 
This may be the most overridden default of Ironic, which means
we need to change the default value.

The default for ``[redfish]use_swift`` was historically ``true``,
however we've generally found that BMCs are particularlly sensitive
to extra characters in the URL as the characters may signify a
dynamically generated file, which would be problematic as virtual
media webservers also generally require range retrieval support.

This change makes the default ``false`` which should lead to one
less override for operators being necessary in practical operation.

Change-Id: Iad57b3c6423bced0e3cb6fb4e31aad6d805f26fa
This commit is contained in:
Julia Kreger 2024-04-15 15:47:27 -07:00
parent 3c5b4cb349
commit ae2b5f5f1f
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ironic/conf/redfish.py
View File

@ -45,7 +45,7 @@ opts = [
default='auto', default='auto',
help=_('Redfish HTTP client authentication method.')), help=_('Redfish HTTP client authentication method.')),
cfg.BoolOpt('use_swift', cfg.BoolOpt('use_swift',
default=True, default=False,
mutable=True, mutable=True,
help=_('Upload generated ISO images for virtual media boot to ' help=_('Upload generated ISO images for virtual media boot to '
'Swift, then pass temporary URL to BMC for booting the ' 'Swift, then pass temporary URL to BMC for booting the '

7
releasenotes/notes/redfish-virtual-media-default-not-swift-3917f0e14fd62a0d.yaml Normal file
View File

@ -0,0 +1,7 @@
---
upgrade:
- |
The default value for ``[redfish]use_swift`` has been changed to
``false``. This is to limit URL validation challenges presented by some
baseboard management controllers where characters in the Swift temporary
URL form are rejected by Baseboard Management Controllers.