openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update cinder from branch 'master'
  to 5ea10b2ecc41ad235cbdbbf94b4a1de98e30e470
  - Merge "Fix test_nvmet_driver tests"
  - Fix test_nvmet_driver tests
    
    Unit tests fails when running on systems without nvmet support.
    
    This is caused by the nvmet library trying to do some safe reads on
    sysfs.  Writtings won't happen though.
    
    This patch ensures that we always use a mocked nvmet library instead of
    trying to use the real library when available like we are doing now.
    
    Example of failure:
    
    cinder.tests.unit.targets.test_nvmet_driver.TestNVMETDriver.test__namespace_dict_shared
    ---------------------------------------------------------------------------------------
    
    Captured traceback:
    ~~~~~~~~~~~~~~~~~~~
        Traceback (most recent call last):
    
          File "/home/zuul/src/code.engineering.redhat.com/cinder/cinder/tests/unit/targets/test_nvmet_driver.py", line 34, in setUp
        self.target = nvmet.NVMET(root_helper=utils.get_root_helper(),
    
          File "/home/zuul/src/code.engineering.redhat.com/cinder/cinder/volume/targets/nvmet.py", line 40, in __init__
        self._nvmet_root = nvmet.Root()
    
          File "/usr/lib/python3.9/site-packages/nvmet/nvme.py", line 241, in __init__
        raise CFSError("%s does not exist.  Giving up." %
    
        nvmet.nvme.CFSError: /sys/kernel/config/nvmet does not exist.  Giving up.
    
    Change-Id: I1845ce07886d7cd6256be55c71e9da3fd9122fe4
This commit is contained in:
Zuul 2023-10-10 01:54:01 +00:00 committed by Gerrit Code Review
parent 79dfa40a26
commit 8f0178652e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder

@ -1 +1 @@
Subproject commit 95630360b2091409dc35eebd86d51d9aad2ab0fc
Subproject commit 5ea10b2ecc41ad235cbdbbf94b4a1de98e30e470