[ironic][release][stable] Ironic Train release can be broken due to entry in driver-requirements.txt
mthode at mthode.org
Fri Oct 18 16:06:09 UTC 2019
On 19-10-18 12:19:12, Dmitry Tantsur wrote:
> Hi all,
> I think we should update global-requirement (on master and train) to
> exclude sushy 1.9.0, like
> Since train has >=1.9.0 currently, it will be a good excuse to change it to
> I'll leave the final word to the stable team though.
> On Wed, Oct 16, 2019 at 3:17 AM <Richard.Pioso at dell.com> wrote:
> > Hi,
> > The Ironic Train release can be broken due to an entry in its
> > driver-requirements.txt. driver-requirements.txt defines a dependency on
> > the sushy package  which can be satisfied by version 1.9.0.
> > Unfortunately, that version contains a few bugs which prevent Ironic from
> > being able to manage Dell EMC and perhaps other vendors' bare metal
> > hardware with its Redfish hardware type (driver). The fixes to them
> >  were merged into master before the creation of stable/train.
> > Therefore, they are available on stable/train and in the last sushy release
> > created during the Train cycle, 2.0.0, the only other version which can
> > satisfy the dependency today. However, consumers -- packagers, operators,
> > and users -- could, fighting time constraints or lacking solid visibility
> > into Ironic, package or install Ironic with sushy 1.9.0 to satisfy the
> > dependency, but, in so doing, unknowingly render the package or
> > installation severely broken.
> > A change  has been proposed as part of a prospective solution to this
> > issue. It creates a new release of sushy from the change which fixes the
> > first bug . Review comments  discuss basing the new release on a more
> > recent stable/train change to pick up other bug fixes and, less
> > importantly, backward compatible feature modifications and enhancements
> > which merged before the change from which 2.0.0 was created. Backward
> > compatible feature modifications and enhancements are interspersed in time
> > among the bug fixes. Once a new release is available, the sushy entry in
> > driver-requirements.txt on stable/train would be updated. However,
> > apparently, the stable branch policy prevents releases from being done at a
> > point earlier than the last release within a given cycle , which was
> > 2.0.0.
> > Another possible resolution which comes to mind is to change the
> > definition of the sushy dependency in driver-requirements.txt  from
> > "sushy>=1.9.0" to "sushy>=2.0.0".
> > Does anyone have a suggestion on how to proceed?
> > Thank you,
> > Rick
> > 
> > https://opendev.org/openstack/ironic/src/commit/b8ae681b37eec617736ac4a507e9a8b3a19e8a58/driver-requirements.txt#L14
> >  https://review.opendev.org/#/c/666253/
> >  https://review.opendev.org/#/c/668936/
> >  https://review.opendev.org/#/c/669889/
> >  https://review.opendev.org/#/c/688551/
> > 
> > https://review.opendev.org/#/c/688551/1/deliverables/train/sushy.yaml@14
Excluding known bad versions to efectively update the minimum constraint
IS allowed by policy as far as I know (from a reqs team perspective).
So this sgtm.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: not available
More information about the openstack-discuss