[Bug 1501838] Re: Tests: lazy load operation of attribute 'snapshot_metadata' cannot proceed

[Corey Bryant]

Hi Tobias, Thanks for the notification on this.  It looks like a
reasonable backport to the liberty cinder package for ubuntu.  We'll
work on getting it backported.  For now, marking this as fix-released
for Xenial+ since both patches landed in 8.0.0.

** Also affects: cinder (Ubuntu Wily)
   Importance: Undecided
       Status: New

** Also affects: cinder (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Changed in: cinder (Ubuntu Wily)
       Status: New => Triaged

** Changed in: cinder (Ubuntu Xenial)
       Status: New => Fix Released

** Changed in: cinder (Ubuntu)
       Status: Confirmed => Fix Released

** Changed in: cinder (Ubuntu)
   Importance: Undecided => High

** Changed in: cinder (Ubuntu Xenial)
   Importance: Undecided => High

** Changed in: cinder (Ubuntu)
   Importance: High => Undecided

** Changed in: cinder (Ubuntu Wily)
   Importance: Undecided => High

** Changed in: cinder (Ubuntu)
   Importance: Undecided => High

-- 
You received this bug notification because you are a member of Ubuntu
OpenStack, which is subscribed to cinder in Ubuntu.
https://bugs.launchpad.net/bugs/1501838

Title:
  Tests: lazy load operation of attribute 'snapshot_metadata' cannot
  proceed

Status in Cinder:
  Fix Released
Status in cinder package in Ubuntu:
  Fix Released
Status in cinder source package in Wily:
  Triaged
Status in cinder source package in Xenial:
  Fix Released

Bug description:
  Possible issue with the Snapshot object?

  master @ 7f498b0 Merge "Obtain target authentication from database
  same as LIO target"

  Running this tox env (subset of our full tests):

  [testenv:py34not]
  commands = ostestr {posargs} --concurrency=16 --regex (cinder.tests.unit.test_volume|cinder.tests.unit.image.test_cache)


  cinder.tests.unit.test_volume.GetActiveByWindowTestCase.test_snapshot_get_active_by_window
  ------------------------------------------------------------------------------------------

  Captured traceback:
  ~~~~~~~~~~~~~~~~~~~
      Traceback (most recent call last):
        File "cinder/tests/unit/test_volume.py", line 5982, in test_snapshot_get_active_by_window
          datetime.datetime(1, 4, 1, 1, 1, 1)).objects
        File "/home/eharney/src/cinder-trunk8/.tox/py34not/lib/python2.7/site-packages/oslo_versionedobjects/base.py", line 171, in wrapper
          result = fn(cls, context, *args, **kwargs)
        File "cinder/objects/snapshot.py", line 251, in get_active_by_window
          snapshots, expected_attrs=['metadata'])
        File "/home/eharney/src/cinder-trunk8/.tox/py34not/lib/python2.7/site-packages/oslo_versionedobjects/base.py", line 991, in obj_make_list
          **extra_args)
        File "cinder/objects/snapshot.py", line 121, in _from_db_object
          metadata = db_snapshot.get('snapshot_metadata')
        File "/home/eharney/src/cinder-trunk8/.tox/py34not/lib/python2.7/site-packages/oslo_db/sqlalchemy/models.py", line 68, in get
          return getattr(self, key, default)
        File "/home/eharney/src/cinder-trunk8/.tox/py34not/lib/python2.7/site-packages/sqlalchemy/orm/attributes.py", line 237, in __get__
          return self.impl.get(instance_state(instance), dict_)
        File "/home/eharney/src/cinder-trunk8/.tox/py34not/lib/python2.7/site-packages/sqlalchemy/orm/attributes.py", line 578, in get
          value = self.callable_(state, passive)
        File "/home/eharney/src/cinder-trunk8/.tox/py34not/lib/python2.7/site-packages/sqlalchemy/orm/strategies.py", line 502, in _load_for_state
          (orm_util.state_str(state), self.key)
      sqlalchemy.orm.exc.DetachedInstanceError: Parent instance <Snapshot at 0x7f38f90ae790> is not bound to a Session; lazy load operation of attribute 'snapshot_metadata' cannot proceed

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