ironic: stop defaulting to iscsi deploy in preparation for its deprecation

We have announced our plans to deprecate and eventually remove the iscsi
deploy interface [1]. To prepare for that, the iscsi deploy is no longer
enabled by default. The default value of IronicImageDownloadSource is changed
to 'http' to reduce the upgrade impact for clouds without swift (ironic
upstream is planning to make a similar change). Underclouds still use swift
for now to save disk space.

[1] http://lists.openstack.org/pipermail/openstack-discuss/2020-September/016952.html

Change-Id: I1de6853eebc545d65ef6ce09276be3bbac2132bd
This commit is contained in:
Dmitry Tantsur 2020-09-02 12:03:58 +02:00 committed by Harald Jensås
parent 4f214d81fe
commit c78f3afa22
3 changed files with 18 additions and 3 deletions

View File

@ -133,7 +133,7 @@ parameters:
type must have at least one valid implementation enabled.
type: comma_delimited_list
IronicEnabledDeployInterfaces:
default: ['iscsi', 'direct']
default: ['direct']
description: Enabled deploy interface implementations. Each hardware
type must have at least one valid implementation enabled.
type: comma_delimited_list
@ -182,7 +182,7 @@ parameters:
description: Whether to enable use of staging drivers.
type: boolean
IronicImageDownloadSource:
default: swift
default: http
description: Image delivery method for the "direct" deploy interface.
Use "swift" for the Object Storage temporary URLs,
use "http" for the local HTTP server (the same as for iPXE).

View File

@ -114,7 +114,7 @@ parameter_defaults:
IronicEnabledHardwareTypes: ['ipmi', 'redfish', 'idrac', 'ilo']
IronicEnabledBootInterfaces: ['pxe', 'ilo-pxe']
IronicEnabledConsoleInterfaces: ['ipmitool-socat', 'ilo', 'no-console']
IronicEnabledDeployInterfaces: ['iscsi', 'direct', 'ansible']
IronicEnabledDeployInterfaces: ['direct', 'ansible']
IronicEnabledInspectInterfaces: ['inspector', 'no-inspect']
IronicEnabledManagementInterfaces: ['ipmitool', 'redfish', 'idrac', 'ilo']
# NOTE(dtantsur): disabling advanced networking as it's not used (or
@ -130,6 +130,8 @@ parameter_defaults:
IronicEnableStagingDrivers: true
IronicCleaningNetwork: 'ctlplane'
IronicForcePowerStateDuringSync: false
# NOTE(dtantsur): remove if/when swift is removed from the undercloud.
IronicImageDownloadSource: swift
IronicInspectorCollectors: default,extra-hardware,numa-topology,logs
IronicInspectorInterface: br-ctlplane
# IronicInspectorSubnets:

View File

@ -0,0 +1,13 @@
---
upgrade:
- |
The ``iscsi`` deploy interface is no longer enabled by default in ironic,
making the ``direct`` deploy interface the default. You will need to
update your nodes to the ``direct`` deploy before upgrading or re-enable
the ``iscsi`` deploy in ``IronicEnabledDeployInterfaces`` (but note that
it is going to be deprecated in the future).
- |
The ``IronicImageDownloadSource`` parameter has been changed to ``http`` by
default making ironic cache glance images and serve them via a local HTTP
server. Set the parameter to ``swift`` to return the previous behavior of
relying on swift temporary URLs.