Open Stack

On Dec 2, 2014, at 7:48 PM, Matt Riedemann <mriedem at linux.vnet.ibm.com> wrote:

> 
> 
> On 12/2/2014 12:33 PM, Ben Nemec wrote:
>> We've discovered a couple of problems as a result of this release.  pep8
>> in most/all of the projects using oslo.concurrency is failing due to the
>> move out of the oslo namespace package and the fact that hacking doesn't
>> know how to handle it, and nova unit tests are failing due to a problem
>> with the way some mocking was done.
>> 
>> Fixes for both of these problems are in progress and should hopefully be
>> available soon.
>> 
>> -Ben
>> 
>> On 12/02/2014 10:29 AM, Ben Nemec wrote:
>>> The Oslo team is pleased to announce the release of oslo.concurrency 0.3.0.
>>> 
>>> This release includes a number of fixes for problems found during the
>>> initial adoptions of the library, as well as some functionality
>>> improvements.
>>> 
>>> For more details, please see the git log history below and
>>> https://launchpad.net/oslo.concurrency/+milestone/0.3.0
>>> 
>>>  Please report issues through launchpad:
>>> https://launchpad.net/oslo.concurrency
>>> 
>>> openstack/oslo.concurrency  0.2.0..HEAD
>>> 
>>> 54c84da Add external lock fixture
>>> 19f07c6 Add a TODO for retrying pull request #20
>>> 46c836e Allow the lock delay to be provided
>>> 3bda65c Allow for providing a customized semaphore container
>>> 656f908 Move locale files to proper place
>>> faa30f8 Flesh out the README
>>> bca4a0d Move out of the oslo namespace package
>>> 58de317 Improve testing in py3 environment
>>> fa52a63 Only modify autoindex.rst if it exists
>>> 63e618b Imported Translations from Transifex
>>> d5ea62c lockutils-wrapper cleanup
>>> 78ba143 Don't use variables that aren't initialized
>>> 
>>>   diffstat (except docs and test files):
>>> 
>>>  .gitignore                                         |   1 +
>>>  .testr.conf                                        |   2 +-
>>>  README.rst                                         |   4 +-
>>>  .../locale/en_GB/LC_MESSAGES/oslo.concurrency.po   |  16 +-
>>>  oslo.concurrency/locale/oslo.concurrency.pot       |  16 +-
>>>  oslo/concurrency/__init__.py                       |  29 ++
>>>  oslo/concurrency/_i18n.py                          |  32 --
>>>  oslo/concurrency/fixture/__init__.py               |  13 +
>>>  oslo/concurrency/fixture/lockutils.py              |  51 --
>>>  oslo/concurrency/lockutils.py                      | 376 --------------
>>>  oslo/concurrency/openstack/__init__.py             |   0
>>>  oslo/concurrency/openstack/common/__init__.py      |   0
>>>  oslo/concurrency/openstack/common/fileutils.py     | 146 ------
>>>  oslo/concurrency/opts.py                           |  45 --
>>>  oslo/concurrency/processutils.py                   | 340 ------------
>>>  oslo_concurrency/__init__.py                       |   0
>>>  oslo_concurrency/_i18n.py                          |  32 ++
>>>  oslo_concurrency/fixture/__init__.py               |   0
>>>  oslo_concurrency/fixture/lockutils.py              |  76 +++
>>>  oslo_concurrency/lockutils.py                      | 502 ++++++++++++++++++
>>>  oslo_concurrency/openstack/__init__.py             |   0
>>>  oslo_concurrency/openstack/common/__init__.py      |   0
>>>  oslo_concurrency/openstack/common/fileutils.py     | 146 ++++++
>>>  oslo_concurrency/opts.py                           |  45 ++
>>>  oslo_concurrency/processutils.py                   | 340 ++++++++++++
>>>  requirements-py3.txt                               |   1 +
>>>  requirements.txt                                   |   1 +
>>>  setup.cfg                                          |   9 +-
>>>  tests/test_lockutils.py                            | 575
>>> ++++++++++++++++++++
>>>  tests/test_processutils.py                         | 519
>>> +++++++++++++++++++
>>>  tests/test_warning.py                              |  29 ++
>>>  tests/unit/__init__.py                             |   0
>>>  tests/unit/test_lockutils.py                       | 543
>>> -------------------
>>>  tests/unit/test_lockutils_eventlet.py              |  59 ---
>>>  tests/unit/test_processutils.py                    | 518 ------------------
>>>  tox.ini                                            |   8 +-
>>>  42 files changed, 3515 insertions(+), 2135 deletions(-)
>>> 
>>>   Requirements updates:
>>> 
>>>  diff --git a/requirements-py3.txt b/requirements-py3.txt
>>>  index b1a8722..a27b434 100644
>>>  --- a/requirements-py3.txt
>>>  +++ b/requirements-py3.txt
>>>  @@ -13,0 +14 @@ six>=1.7.0
>>>  +retrying>=1.2.2,!=1.3.0 # Apache-2.0
>>>  diff --git a/requirements.txt b/requirements.txt
>>>  index b1a8722..a27b434 100644
>>>  --- a/requirements.txt
>>>  +++ b/requirements.txt
>>>  @@ -13,0 +14 @@ six>=1.7.0
>>>  +retrying>=1.2.2,!=1.3.0 # Apache-2.0
>>> 
>> 
>> 
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>> 
> 
> Was a bug reported for the nova unit tests that were mocking out external_lock in lockutils?  I didn't see one so I opened a bug [1] and wrote the elastic-recheck query against that.  I'm working on fixing the tests in the meantime but I'll gladly stop if someone else has a fix up for review.
> 
> [1] https://bugs.launchpad.net/nova/+bug/1398624

Ben has a patch up to use the fixtures in the library instead: https://review.openstack.org/138463

> 
> -- 
> 
> Thanks,
> 
> Matt Riedemann
> 
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev