[openstack-dev] We should move strutils.mask_password back into oslo-incubator
Matt Riedemann
mriedem at linux.vnet.ibm.com
Wed Oct 7 19:38:07 UTC 2015
Here's why:
https://review.openstack.org/#/c/220622/
That's marked as fixing an OSSA which means we'll have to backport the
fix in nova but it depends on a change to strutils.mask_password in
oslo.utils, which required a release and a minimum version bump in
global-requirements.
To backport the change in nova, we either have to:
1. Copy mask_password out of oslo.utils and add it to nova in the
backport or,
2. Backport the oslo.utils change to a stable branch, release it as a
patch release, bump minimum required version in stable g-r and then
backport the nova change and depend on the backported oslo.utils stable
release - which also makes it a dependent library version bump for any
packagers/distros that have already frozen libraries for their stable
releases, which is kind of not fun.
So I'm thinking this is one of those things that should ultimately live
in oslo-incubator so it can live in the respective projects. If
mask_password were in oslo-incubator, we'd have just fixed and
backported it there and then synced to nova on master and stable
branches, no dependent library version bumps required.
Plus I miss the good old days of reviewing oslo-incubator
syncs...(joking of course).
--
Thanks,
Matt Riedemann
More information about the OpenStack-dev
mailing list