yahoo-eng-team team mailing list archive

[OpenStack Infra <2043116@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/nova/+/900532
Committed: https://opendev.org/openstack/nova/commit/e04d20a8bc3cfea11c9699219919d0a0158d74fb
Submitter: "Zuul (22348)"
Branch:    master

commit e04d20a8bc3cfea11c9699219919d0a0158d74fb
Author: Takashi Kajinami <kajinamit@xxxxxxxxxxxxxxx>
Date:   Thu Nov 9 23:58:39 2023 +0900

    Fix unit tests broken by olso.utils
    
    Some of the object unit tests grub Mock object unintentionally, and
    that results in failure during initializing an versioned object,
    because the Mock object does not present its version correctly.
    
    This fixes that problem
    
    The sqlalchemy-2x job is made non-voting because this job requires
    oslo.utils 6.3.0 which is blocked by this problem.
    
    Closes-Bug: #2043116
    Related-Bug: #2042886
    Change-Id: Id1c1b77c020e2d8f0ac61c2aee86c9f37d388d53


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/2043116

Title:
  Unit test fails with oslo.utils 6.3.0

Status in Ironic:
  Fix Released
Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  Description
  ===========

  We recently created oslo.utils 6.3.0 release which includes the fix
  for the bug[1] affecting sqlalchemy-master jobs.

  However when we attempt to bump the version in u-c file[2], unit tests
  of ironic and nova fail with the following errors.

  ```
  nova.tests.unit.objects.test_objects.TestRegistry.test_hook_keeps_newer_properly
  --------------------------------------------------------------------------------

  Captured traceback:
  ~~~~~~~~~~~~~~~~~~~
      Traceback (most recent call last):

        File "/usr/lib/python3.10/unittest/mock.py", line 1379, in patched
      return func(*newargs, **newkeywargs)

        File "/home/zuul/src/opendev.org/openstack/nova/nova/tests/unit/objects/test_objects.py", line 1060, in test_hook_keeps_newer_properly
      reg.registration_hook(MyObj, 0)

        File "/home/zuul/src/opendev.org/openstack/nova/nova/objects/base.py", line 72, in registration_hook
      cur_version = versionutils.convert_version_to_tuple(

        File "/home/zuul/src/opendev.org/openstack/nova/.tox/py310/lib/python3.10/site-packages/oslo_utils/versionutils.py", line 91, in convert_version_to_tuple
      version_str = re.sub(r'(\d+)(a|alpha|b|beta|rc)\d+$', '\\1', version_str)

        File "/usr/lib/python3.10/re.py", line 209, in sub
      return _compile(pattern, flags).sub(repl, string, count)

      TypeError: expected string or bytes-like object
  ```

  [1] https://bugs.launchpad.net/oslo.utils/+bug/2042886
  [2] https://review.opendev.org/c/openstack/requirements/+/900517

To manage notifications about this bug go to:
https://bugs.launchpad.net/ironic/+bug/2043116/+subscriptions