| 2017-04-07 13:23:24 |
Jon Bernard |
description |
This test 'test_volume_backup_create_get_detailed_list_restore_delete' was recently changed [1] to pass a non-default container parameter to create_backup(). For the Ceph backend, this functionality is not supported and a quick fix [2] was proposed to skip the entire test for Ceph. By skipping the entire test, we do loose some test coverage for the ceph backend for the case there the default container is used. A finer-grained approach is preferable where either the test is broken into separate pieces or the container= and its assert() are conditionally skipped for backends which do not support it. |
This test 'test_volume_backup_create_get_detailed_list_restore_delete' was recently changed [1] to pass a non-default container parameter to create_backup(). For the Ceph backend, this functionality is not supported and a quick fix [2] was proposed to skip the entire test for Ceph. By skipping the entire test, we do loose some test coverage for the ceph backend for the case there the default container is used. A finer-grained approach is preferable where either the test is broken into separate pieces or the container= and its assert() are conditionally skipped for backends which do not support it.
[1] https://review.openstack.org/#/c/453140/
[2] https://review.openstack.org/#/c/454321/ |
|
