From b2a249d186b77def6cd449d34fd600520c159a50 Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Thu, 24 Jun 2021 09:31:57 -0700 Subject: Deprecate [pxe]ip_version parameter The parameter has not had any effect in the code since the dual stack PXE work went into place with Ironic as options are now generated and transmitted to Neutron for both IPv4 and IPv6. This option is only used by the internal ``dhcp_options_for_instance`` method in ``ironic.common.pxe_utils`` as a fallback if the calling method does not specify a version. However a later change resulted in the default behavior calling the dhcp option generation explicitly to generate both IPv4 and IPv6 parameters, making the option entirely redundant. Third party drivers using the ``dhcp_options_for_instance`` method should consider generating options for both IPv4 and IPv6 at all times. See change If7a296001e204ae0c9a49495731052ab33379628 for examples on how to do this. Change-Id: I343783389105f008ce6dafc8d25d93211710771a --- devstack/lib/ironic | 3 --- 1 file changed, 3 deletions(-) (limited to 'devstack') diff --git a/devstack/lib/ironic b/devstack/lib/ironic index ebe77cc57..be692de98 100644 --- a/devstack/lib/ironic +++ b/devstack/lib/ironic @@ -1465,9 +1465,6 @@ function configure_ironic { iniset_rpc_backend ironic $IRONIC_CONF_FILE fi - # Set IP version - iniset $IRONIC_CONF_FILE pxe ip_version $IRONIC_IP_VERSION - # Configure Ironic conductor, if it was enabled. if is_service_enabled ir-cond; then configure_ironic_conductor -- cgit v1.2.1