Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-09 Thread Doug Hellmann
Excerpts from Matt Riedemann's message of 2015-10-09 09:51:28 -0500: > > On 10/9/2015 1:49 AM, Paul Carlton wrote: > > > > On 08/10/15 16:49, Doug Hellmann wrote: > >> Excerpts from Matt Riedemann's message of 2015-10-07 14:38:07 -0500: > >>> Here's why: > >>> > >>> https://review.openstack.org/#/

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-09 Thread Matt Riedemann
On 10/9/2015 1:49 AM, Paul Carlton wrote: On 08/10/15 16:49, Doug Hellmann wrote: Excerpts from Matt Riedemann's message of 2015-10-07 14:38:07 -0500: 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

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Paul Carlton
On 08/10/15 16:49, Doug Hellmann wrote: Excerpts from Matt Riedemann's message of 2015-10-07 14:38:07 -0500: 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_pass

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Doug Hellmann
Excerpts from Matt Riedemann's message of 2015-10-07 14:38:07 -0500: > 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 re

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Ihar Hrachyshka
> On 08 Oct 2015, at 16:51, Matt Riedemann wrote: > > > > On 10/8/2015 9:25 AM, Jeremy Stanley wrote: >> On 2015-10-08 08:58:06 -0500 (-0500), Matt Riedemann wrote: >> [...] >>> I don't know how many operators are tracking patch releases of >>> dependencies on stable branches unless there is a

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Matt Riedemann
On 10/8/2015 9:25 AM, Jeremy Stanley wrote: On 2015-10-08 08:58:06 -0500 (-0500), Matt Riedemann wrote: [...] I don't know how many operators are tracking patch releases of dependencies on stable branches unless there is a new minimum requirement on those, especially if they aren't getting the

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Jeremy Stanley
On 2015-10-08 08:58:06 -0500 (-0500), Matt Riedemann wrote: [...] > I don't know how many operators are tracking patch releases of > dependencies on stable branches unless there is a new minimum > requirement on those, especially if they aren't getting their > updates from a distro provider. So whi

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Matt Riedemann
On 10/8/2015 4:21 AM, Julien Danjou wrote: On Wed, Oct 07 2015, Matt Riedemann wrote: 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 stab

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-08 Thread Julien Danjou
On Wed, Oct 07 2015, Matt Riedemann wrote: > 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 de

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-07 Thread Matt Riedemann
On 10/7/2015 6:00 PM, Robert Collins wrote: On 8 October 2015 at 08:38, Matt Riedemann wrote: 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.

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-07 Thread Robert Collins
On 8 October 2015 at 08:38, Matt Riedemann wrote: > 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

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-07 Thread Joshua Harlow
Isn't #2 the right approach? Even if it might be more 'work' I personally would prefer #2 and do it right (and make it easier to do the right thing in the future via scripts, automation, other) vs. the other mentioned approaches. If we were consuming, say a 3rd party library and that 3rd part

Re: [openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-07 Thread Davanum Srinivas
Matt, My vote is for #1, as we should kill oslo-incubator in Mitaka. Thanks, Dims On Wed, Oct 7, 2015 at 3:38 PM, Matt Riedemann wrote: > 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 dep

[openstack-dev] We should move strutils.mask_password back into oslo-incubator

2015-10-07 Thread Matt Riedemann
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