Project

General

Profile

Bug #41907

fix pytest warnings

Added by Rishabh Dave 4 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
nautilus, mimic, luminous
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature:

Description

Following two warning are thrown a bunch of times when running pytest

ceph_volume/tests/test_configuration.py::TestConf::test_get_non_existing_list
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:29: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
    cfg = configuration.Conf()

ceph_volume/tests/test_configuration.py::TestConf::test_get_non_existing_list
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:31: DeprecationWarning: This method will be removed in future versions.  Use 'parser.read_file()' instead.
    cfg.readfp(self.conf_file)

ceph_volume/tests/test_configuration.py::TestConf::test_get_non_existing_list_get_default
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:35: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
    cfg = configuration.Conf()

ceph_volume/tests/test_configuration.py::TestConf::test_get_non_existing_list_get_default
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:37: DeprecationWarning: This method will be removed in future versions.  Use 'parser.read_file()' instead.
    cfg.readfp(self.conf_file)

ceph_volume/tests/test_configuration.py::TestConf::test_get_rid_of_comments
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:41: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
    cfg = configuration.Conf()

ceph_volume/tests/test_configuration.py::TestConf::test_get_rid_of_comments
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:48: DeprecationWarning: This method will be removed in future versions.  Use 'parser.read_file()' instead.
    cfg.readfp(conf_file)

ceph_volume/tests/test_configuration.py::TestConf::test_gets_split_on_commas
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:52: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
    cfg = configuration.Conf()

ceph_volume/tests/test_configuration.py::TestConf::test_gets_split_on_commas
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:59: DeprecationWarning: This method will be removed in future versions.  Use 'parser.read_file()' instead.
    cfg.readfp(conf_file)

ceph_volume/tests/test_configuration.py::TestConf::test_spaces_and_tabs_are_ignored
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:63: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
    cfg = configuration.Conf()

ceph_volume/tests/test_configuration.py::TestConf::test_spaces_and_tabs_are_ignored
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/tests/test_configuration.py:70: DeprecationWarning: This method will be removed in future versions.  Use 'parser.read_file()' instead.
    cfg.readfp(conf_file)

ceph_volume/tests/test_configuration.py::TestLoad::test_load_from_path
ceph_volume/tests/test_configuration.py::TestLoad::test_load_with_colon_comments
ceph_volume/tests/test_configuration.py::TestLoad::test_load_with_hash_comments
ceph_volume/tests/test_configuration.py::TestLoad::test_unable_to_read_configuration
ceph_volume/tests/test_configuration.py::TestLoad::test_coment_as_a_value[colon]
ceph_volume/tests/test_configuration.py::TestLoad::test_coment_as_a_value[hash]
ceph_volume/tests/devices/lvm/test_prepare.py::TestGetClusterFsid::test_fsid_is_read_from_ceph_conf
ceph_volume/tests/util/test_encryption.py::TestDmcryptKey::test_dmcrypt_with_default_size
ceph_volume/tests/util/test_encryption.py::TestDmcryptKey::test_dmcrypt_with_custom_size
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_default_options
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_multiple_options_are_used
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_multiple_options_will_get_the_force_flag
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_underscore_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_default_options
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_mount_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_multiple_whitespace_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_multiple_comma_whitespace_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_underscore_mount_options_are_used
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_undefined_size_fallbacks_formatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_undefined_size_fallbacks_unformatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_defined_size_unformatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_defined_size_formatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_refuse_tiny_journals
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/configuration.py:47: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
    parser = Conf()

ceph_volume/tests/test_configuration.py::TestLoad::test_load_from_path
ceph_volume/tests/test_configuration.py::TestLoad::test_load_with_colon_comments
ceph_volume/tests/test_configuration.py::TestLoad::test_load_with_hash_comments
ceph_volume/tests/test_configuration.py::TestLoad::test_unable_to_read_configuration
ceph_volume/tests/test_configuration.py::TestLoad::test_coment_as_a_value[colon]
ceph_volume/tests/test_configuration.py::TestLoad::test_coment_as_a_value[hash]
ceph_volume/tests/devices/lvm/test_prepare.py::TestGetClusterFsid::test_fsid_is_read_from_ceph_conf
ceph_volume/tests/util/test_encryption.py::TestDmcryptKey::test_dmcrypt_with_default_size
ceph_volume/tests/util/test_encryption.py::TestDmcryptKey::test_dmcrypt_with_custom_size
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_default_options
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_multiple_options_are_used
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_multiple_options_will_get_the_force_flag
ceph_volume/tests/util/test_prepare.py::TestFormatDevice::test_underscore_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_default_options
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_mount_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_multiple_whitespace_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_multiple_comma_whitespace_options_are_used
ceph_volume/tests/util/test_prepare.py::TestMountOSD::test_underscore_mount_options_are_used
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_undefined_size_fallbacks_formatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_undefined_size_fallbacks_unformatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_defined_size_unformatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_defined_size_formatted
ceph_volume/tests/util/test_prepare.py::TestGetJournalSize::test_refuse_tiny_journals
  /home/fedora/repos/ceph/master/src/ceph-volume/ceph_volume/configuration.py:53: DeprecationWarning: This method will be removed in future versions.  Use 'parser.read_file()' instead.
    parser.readfp(trimmed_conf)


Related issues

Copied to ceph-volume - Backport #42048: mimic: fix pytest warnings Resolved
Copied to ceph-volume - Backport #42049: luminous: fix pytest warnings Resolved
Copied to ceph-volume - Backport #42050: nautilus: fix pytest warnings Resolved

History

#1 Updated by Rishabh Dave 4 months ago

  • Status changed from New to Fix Under Review

#2 Updated by Jan Fajerski 4 months ago

  • Status changed from Fix Under Review to Pending Backport
  • Backport set to nautilus, mimic, luminous

#3 Updated by Nathan Cutler 4 months ago

#4 Updated by Nathan Cutler 4 months ago

#5 Updated by Nathan Cutler 4 months ago

#6 Updated by Jan Fajerski 4 months ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF