[dev][stable][release][manila] Should we revert these stable branch backports?
tpb at dyncloud.net
Wed Mar 27 12:56:54 UTC 2019
In manila we recently merged backports of a change  that aims
to fix up faulty configuration option deprecations in the
Dell-EMC VMAX driver. The question at hand is whether (a) we
should revert these backports on the grounds that the
deprecations were faulty and therefore are only effective from
Stein forwards, or whether (b) the deprecations actually worked
and took effect back when Ocata was the development branch but
had faults that should be corrected all the way back to Pike
before it goes EM.
I think (b) is the correct answer but let's check.
On Jan 10 2017 a review  merged that deprecated generic
Dell-EMC driver options in favor of model-specific options .
There were two models at the time, Unity and VNX. This change
was in itself unproblematic.
On Jan 24 2017 a review  merged that introduced a third
Dell-EMC model, VMAX. This new code introduced VMAX specific
options, consistent with the deprecation of the generic Dell-EMC
options. However it had two problems which review  corrects.
When it defined the new VMAX-specific options it failed to
indicate the corresponding old generic options via
'deprecated_name' . Worse, the code that consumed the options
actually looked for the old generic 'emc_interface_ports' option
instead of the new 'vmax_ethernet_ports' option. The only way to
set a value for this option was to use its deprecated
The change that we backported fixes both of these problems.
I think it is a valid stable backport.
If people set this "ports" option using the deprecated option it
will still work. The deprecated option has not been removed yet
and will never be removed from any of the stable branches.
If they tried to set it via the proper option then they would have
had a functional problem, a bug, which this change now fixes.
If on the other hand we say we cannot backport this then until
stable/stein there will be no way for VMAX users of earlier stable
branches to set this option except by magic -- by using an old
deprecated form of the option that without the backported fix is not
even visible in the customer-facing configuration for VMAX.
All this said, I could well be missing something so I welcome
analysis by stable/core folks. The backports in question have not
yet been released so it would not be a problem to revert them.
-- Tom Barron
 If this was the only issue then arguably it wasn't an error
since the VMAX driver never operated with the old options.
More information about the openstack-discuss