Testcase failed due to real ceph cluster

Bug #1909109 reported by zhaoleilc
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Triaged
Low
Eric Harney
Victoria
Triaged
Low
Unassigned
Wallaby
Triaged
Low
Eric Harney

Bug Description

Description
===========
I cloned the cinder repository of victoria branch and ran
tox tests. Unfortunately, the testcase which is 'cinder.tests.unit.volume.
drivers.test_rbd.RBDTestCase.test__set_keyring_attributes_cinderlib'
failed. This testcase was affected by the real ceph cluster which
had already produced the file whose path is '/etc/ceph/ceph.client.
admin.keyring'. However, testcases should be isolated instead of
affecting by their execution environments.

Steps to reproduce
==================
1. Deploy a ceph cluster on a node and make sure /etc/ceph/ceph.client.
admin.keyring exits.
2. Clone the cinder repository of victoria branch and ran
command such as 'tox -e py3'

Expected result
===============
All testcases succeed.

Actual result
=============
The testcase which is 'cinder.tests.unit.volume.
drivers.test_rbd.RBDTestCase.test__set_keyring_attributes_cinderlib'
failed.

Logs & Configs
==============
==============================
Failed 1 tests - output below:
==============================

cinder.tests.unit.volume.drivers.test_rbd.RBDTestCase.test__set_keyring_attributes_cinderlib
--------------------------------------------------------------------------------------------

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

      File "/root/gitlab/cinder/cinder/tests/unit/volume/drivers/test_rbd.py", line 1638, in test__set_keyring_attributes_cinderlib
    self.assertIsNone(self.driver.keyring_data)

      File "/root/gitlab/cinder/.tox/py3/lib/python3.6/site-packages/testtools/testcase.py", line 430, in assertIsNone
    self.assertThat(observed, matcher, message)

      File "/root/gitlab/cinder/.tox/py3/lib/python3.6/site-packages/testtools/testcase.py", line 502, in assertThat
    raise mismatch_error

    testtools.matchers._impl.MismatchError: is not:
reference = None
actual = '''\
[client.admin]
\tkey = AQAEFtRfl81/CRAAkC9xP07CKa6d/q+3tNErWQ==
\tcaps mds = "allow *"
\tcaps mgr = "allow *"
\tcaps mon = "allow *"
\tcaps osd = "allow *"
'''

Tags: ceph unit-test
tags: added: ceph unit-test
Changed in cinder:
status: New → Triaged
importance: Undecided → Low
Eric Harney (eharney)
Changed in cinder:
assignee: nobody → Eric Harney (eharney)
Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

Issue was introduced by commit be02e0f3467b8cac in victoria

Changed in cinder:
milestone: none → wallaby-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 18.0.0.0b1

This issue was fixed in the openstack/cinder 18.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 17.1.0

This issue was fixed in the openstack/cinder 17.1.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.