Open Stack

> On Jan 13, 2015, at 12:12 PM, Sean Dague <sean at dague.net> wrote:
> 
> On 01/13/2015 11:46 AM, Matt Riedemann wrote:
>> 
>> 
>> On 1/13/2015 10:21 AM, Matt Riedemann wrote:
>>> 
>>> 
>>> On 1/13/2015 9:07 AM, Doug Hellmann wrote:
>>>> The Oslo team is pleased to announce the release of
>>>> oslo.concurrency 0.4.0: oslo.concurrency library
>>>> 
>>>> This release removes the use of ConfigFilter when accessing
>>>> configuration options, making it possible for applications
>>>> to set defaults for the lock path option that refer to other
>>>> option values.
>>>> 
>>>> This version also adds a new context manager for detecting
>>>> long-running operations and logging when they take longer
>>>> than expected. See oslo_concurrency.watchdog.
>>>> 
>>>> For more details, please see the git log history below and
>>>>  http://launchpad.net/oslo.concurrency/+milestone/0.4.0
>>>> 
>>>> Please report issues through launchpad:
>>>>  http://bugs.launchpad.net/oslo.concurrency
>>>> 
>>>> ----------------------------------------
>>>> 
>>>> Changes in openstack/oslo.concurrency  0.3.0..0.4.0
>>>> 
>>>> 3dfefe6 Bump to hacking 0.10
>>>> 43dc67e Updated from global requirements
>>>> df35680 add watchdog module
>>>> 18bcbe2 Updated from global requirements
>>>> 234b007 make time format for processutils match lockutils
>>>> a414266 Correct the translation domain for loading messages
>>>> 9066336 Add a reader/writer lock
>>>> 9dd6d21 Don't use ConfigFilter for lockutils
>>>> 093ed4f Report import warnings where the import occurs
>>>> 7c7493f Port processutils to Python 3
>>>> 2aafe6f Activate pep8 check that _ is imported
>>>> 32bf940 Drop requirements-py3.txt
>>>> deb0152 Updated from global requirements
>>>> d59543d Clean up API documentation
>>>> 5de5c42 Workflow documentation is now in infra-manual
>>>> 09ab853 Remove noqa from test files
>>>> 3de55f3 test compatibility for old imports
>>>> 4fd64cb Fix bug link in README.rst
>>>> 
>>>>   diffstat (except docs and test files):
>>>> 
>>>>  .gitignore                                       |   1 -
>>>>  CONTRIBUTING.rst                                 |   7 +-
>>>>  README.rst                                       |   2 +-
>>>>  oslo/concurrency/__init__.py                     |   2 +-
>>>>  oslo_concurrency/_i18n.py                        |   2 +-
>>>>  oslo_concurrency/lockutils.py                    | 175 +++++++++++-
>>>>  oslo_concurrency/opts.py                         |   2 +-
>>>>  oslo_concurrency/processutils.py                 |  23 +-
>>>>  oslo_concurrency/watchdog.py                     |  66 +++++
>>>>  requirements-py3.txt                             |  14 -
>>>>  requirements.txt                                 |   6 +-
>>>>  setup.cfg                                        |   5 +-
>>>>  test-requirements.txt                            |   3 +-
>>>>  tests/__init__.py                                |   2 +-
>>>>  tests/test_import.py                             |  31 +++
>>>>  tests/test_lockutils.py                          |   4 +-
>>>>  tests/test_processutils.py                       |  18 +-
>>>>  tests/test_warning.py                            |  32 +++
>>>>  tests/test_watchdog.py                           |  75 ++++++
>>>>  tox.ini                                          |   5 -
>>>>  31 files changed, 832 insertions(+), 90 deletions(-)
>>>> 
>>>>   Requirements updates:
>>>> 
>>>>  diff --git a/requirements.txt b/requirements.txt
>>>>  index a27b434..fb89633 100644
>>>>  --- a/requirements.txt
>>>>  +++ b/requirements.txt
>>>>  @@ -9 +9 @@ fixtures>=0.3.14
>>>>  -oslo.config>=1.4.0  # Apache-2.0
>>>>  +oslo.config>=1.6.0  # Apache-2.0
>>>>  @@ -11 +11 @@ oslo.i18n>=1.0.0  # Apache-2.0
>>>>  -oslo.utils>=1.0.0                       # Apache-2.0
>>>>  +oslo.utils>=1.2.0                       # Apache-2.0
>>>>  @@ -14 +14 @@ six>=1.7.0
>>>>  -retrying>=1.2.2,!=1.3.0 # Apache-2.0
>>>>  +retrying>=1.2.3,!=1.3.0 # Apache-2.0
>>>>  diff --git a/test-requirements.txt b/test-requirements.txt
>>>>  index c424655..32cdaae 100644
>>>>  --- a/test-requirements.txt
>>>>  +++ b/test-requirements.txt
>>>>  @@ -5 +5 @@
>>>>  -hacking>=0.9.1,<0.10
>>>>  +hacking>=0.10.0,<0.11
>>>>  @@ -7,0 +8 @@ coverage>=3.6
>>>>  +futures>=2.1.6
>>>> 
>>>> __________________________________________________________________________
>>>> 
>>>> 
>>>> OpenStack Development Mailing List (not for usage questions)
>>>> Unsubscribe:
>>>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>> 
>>> 
>>> Looks like we probably have a new bug in nova now:
>>> 
>>> http://goo.gl/i7qa0f
>>> 
>> 
>> Bug is reported with details inline:
>> 
>> https://bugs.launchpad.net/oslo.concurrency/+bug/1410348
> 
> Here is a way to address if from purely the nova side -
> https://review.openstack.org/146929

Patch to oslo.concurrency: https://review.openstack.org/#/c/146932/
Patch to nova to use the above: https://review.openstack.org/#/c/146937/

My nova patch would require a new release of oslo.concurrency, which we’ll be making as quickly as possible, but in the mean time it makes sense to merge Sean’s patch to nova.

Doug

> 
> 	-Sean
> 
> -- 
> Sean Dague
> http://dague.net
> 
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev