Re: [openstack-dev] oslo.concurrency 0.3.0 released

2014-12-03 Thread Doug Hellmann

On Dec 2, 2014, at 7:48 PM, Matt Riedemann mrie...@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@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.
 
 

Re: [openstack-dev] oslo.concurrency 0.3.0 released

2014-12-02 Thread Ben Nemec
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@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] oslo.concurrency 0.3.0 released

2014-12-02 Thread Doug Hellmann
One of the changes in this release was a move from using the “oslo” namespace 
to using a non-namespaced package “oslo_concurrency”. We included some shims to 
allow imports to work correctly, but the hacking rule to verify whether an 
import is actually a module was not recognizing these shims correctly and so a 
whole lot of test jobs failed. We are sorry for the inconvenience.

A new version of hacking, 0.9.4, has been released with a fix for this problem. 
If your pep8 tests failed, you should recheck them when the current 
infrastructure mirror issue is resolved. If you have a local tox environment, 
you can rebuild it with “tox -e pep8 -r” to install the new version of hacking 
and verify that it works before rechecking on the shared CI systems.

We will be providing more guidance on the namespace package change when more of 
the Oslo libraries are updated. If you’re curious, you can check out 
http://specs.openstack.org/openstack/oslo-specs/specs/kilo/drop-namespace-packages.html
 now.

Thanks,
Doug


On Dec 2, 2014, at 11:29 AM, Ben Nemec openst...@nemebean.com 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
 
 ___
 

Re: [openstack-dev] oslo.concurrency 0.3.0 released

2014-12-02 Thread Matt Riedemann



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@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

--

Thanks,

Matt Riedemann


___