From ae2b5f5f1f2dbac3150bb37bfba395474f7d08da Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Mon, 15 Apr 2024 15:47:27 -0700 Subject: [PATCH] redfish: change default virtual media storage to local storage 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 --- ironic/conf/redfish.py | 2 +- ...h-virtual-media-default-not-swift-3917f0e14fd62a0d.yaml | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/redfish-virtual-media-default-not-swift-3917f0e14fd62a0d.yaml diff --git a/ironic/conf/redfish.py b/ironic/conf/redfish.py index 87a359d097..7ef0ff4b61 100644 --- a/ironic/conf/redfish.py +++ b/ironic/conf/redfish.py @@ -45,7 +45,7 @@ opts = [ default='auto', help=_('Redfish HTTP client authentication method.')), cfg.BoolOpt('use_swift', - default=True, + default=False, mutable=True, help=_('Upload generated ISO images for virtual media boot to ' 'Swift, then pass temporary URL to BMC for booting the ' diff --git a/releasenotes/notes/redfish-virtual-media-default-not-swift-3917f0e14fd62a0d.yaml b/releasenotes/notes/redfish-virtual-media-default-not-swift-3917f0e14fd62a0d.yaml new file mode 100644 index 0000000000..add8df608a --- /dev/null +++ b/releasenotes/notes/redfish-virtual-media-default-not-swift-3917f0e14fd62a0d.yaml @@ -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.