)]}'
{"cinder/backup/drivers/ceph.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"93761f0aa7595d721d0f2222ae0e00ae5ea3936d","unresolved":false,"context_lines":[{"line_number":99,"context_line":"]"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"CONF \u003d cfg.CONF"},{"line_number":102,"context_line":"CONF.register_opts(CEPH_BACKUP_OPTS,"},{"line_number":103,"context_line":"                   group\u003dconfiguration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"9f560f44_a48ba61b","line":102,"updated":"2020-10-12 19:36:36.000000000","message":"This doesn\u0027t works with tools/config/generate_cinder_opts.py too:(","commit_id":"c2e173ec7109ddcc405aba7bb13d015a31fdd603"}],"cinder/backup/drivers/glusterfs.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        backup_path \u003d self._init_backup_repo_path()"},{"line_number":55,"context_line":"        super(GlusterfsBackupDriver, self).__init__(context,"},{"line_number":56,"context_line":"                                                    backup_path\u003dbackup_path,"},{"line_number":57,"context_line":"                                                    configuration\u003dconfiguration)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    @staticmethod"},{"line_number":60,"context_line":"    def get_driver_options():"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_49edc689","line":57,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        backup_path \u003d self._init_backup_repo_path()"},{"line_number":55,"context_line":"        super(GlusterfsBackupDriver, self).__init__(context,"},{"line_number":56,"context_line":"                                                    backup_path\u003dbackup_path,"},{"line_number":57,"context_line":"                                                    configuration\u003dconfiguration)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    @staticmethod"},{"line_number":60,"context_line":"    def get_driver_options():"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_a8f5b7ef","line":57,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        backup_path \u003d self._init_backup_repo_path()"},{"line_number":55,"context_line":"        super(GlusterfsBackupDriver, self).__init__(context,"},{"line_number":56,"context_line":"                                                    backup_path\u003dbackup_path,"},{"line_number":57,"context_line":"                                                    configuration\u003dconfiguration)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    @staticmethod"},{"line_number":60,"context_line":"    def get_driver_options():"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_4a78968e","line":57,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        backup_path \u003d self._init_backup_repo_path()"},{"line_number":55,"context_line":"        super(GlusterfsBackupDriver, self).__init__(context,"},{"line_number":56,"context_line":"                                                    backup_path\u003dbackup_path,"},{"line_number":57,"context_line":"                                                    configuration\u003dconfiguration)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    @staticmethod"},{"line_number":60,"context_line":"    def get_driver_options():"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_79ff5600","line":57,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"}],"cinder/backup/manager.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"599659abeba8b9869f99fad4d1f6c50a302f5eb8","unresolved":false,"context_lines":[{"line_number":98,"context_line":"]"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"CONF \u003d cfg.CONF"},{"line_number":101,"context_line":"CONF.register_opts(["},{"line_number":102,"context_line":"    cfg.IntOpt(\u0027backup_driver_stats_polling_interval\u0027,"},{"line_number":103,"context_line":"               default\u003d60,"},{"line_number":104,"context_line":"               min\u003d10,"}],"source_content_type":"text/x-python","patch_set":11,"id":"9f560f44_248e7631","line":101,"updated":"2020-10-12 19:35:42.000000000","message":"This line breaks tools/config/generate_cinder_opts.py","commit_id":"c2e173ec7109ddcc405aba7bb13d015a31fdd603"}],"cinder/cmd/backup.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    #    _launch_backup_process(launcher, i, semaphore)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    try:"},{"line_number":124,"context_line":"#        import pdb;pdb.set_trace()"},{"line_number":125,"context_line":"        for backend in filter(None, CONF.enabled_backup_backends):"},{"line_number":126,"context_line":"            _launch_backup_process(launcher, backend, semaphore)"},{"line_number":127,"context_line":"    except TypeError:"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_192d7a8e","line":124,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E115 expected an indented block (comment)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"}],"cinder/tests/unit/backup/drivers/test_backup_ceph.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":39,"context_line":"from cinder import objects"},{"line_number":40,"context_line":"from cinder.tests.unit import fake_constants as fake"},{"line_number":41,"context_line":"from cinder.volume import configuration as conf"},{"line_number":42,"context_line":"from cinder.tests.unit import test"},{"line_number":43,"context_line":"import cinder.volume.drivers.rbd as rbd_driver"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"# This is used to collect raised exceptions so that tests may check what was"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_c8f2c306","line":42,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: H306: imports not in alphabetical order (cinder.volume.configuration, cinder.tests.unit.test)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":38,"context_line":"from cinder import objects"},{"line_number":39,"context_line":"from cinder.tests.unit import fake_constants as fake"},{"line_number":40,"context_line":"from cinder.volume import configuration as conf"},{"line_number":41,"context_line":"from cinder.tests.unit import test"},{"line_number":42,"context_line":"import cinder.volume.drivers.rbd as rbd_driver"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"# This is used to collect raised exceptions so that tests may check what was"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_0ab33e4f","line":41,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: H306: imports not in alphabetical order (cinder.volume.configuration, cinder.tests.unit.test)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":38,"context_line":"from cinder import objects"},{"line_number":39,"context_line":"from cinder.tests.unit import fake_constants as fake"},{"line_number":40,"context_line":"from cinder.volume import configuration as conf"},{"line_number":41,"context_line":"from cinder.tests.unit import test"},{"line_number":42,"context_line":"import cinder.volume.drivers.rbd as rbd_driver"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"# This is used to collect raised exceptions so that tests may check what was"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_3932fef2","line":41,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: H306: imports not in alphabetical order (cinder.volume.configuration, cinder.tests.unit.test)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"}],"cinder/tests/unit/backup/test_backup.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_a9056a48","line":62,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E225 missing whitespace around operator","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_89022e3c","line":62,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (109 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":238,"context_line":"        def get_admin_context():"},{"line_number":239,"context_line":"            return self.ctxt"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":242,"context_line":"        self.override_config(\u0027periodic_interval\u0027, 0, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"        vol1_id \u003d self._create_volume_db_entry()"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_e9fbf243","line":241,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":239,"context_line":"            return self.ctxt"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":242,"context_line":"        self.override_config(\u0027periodic_interval\u0027, 0, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"        vol1_id \u003d self._create_volume_db_entry()"},{"line_number":245,"context_line":"        self._create_volume_attach(vol1_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_c9f8b64a","line":242,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (92 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":345,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name)"},{"line_number":348,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":349,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":350,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"},{"line_number":351,"context_line":"        mock_FILC.return_value \u003d mock_init_loop"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_29979aee","line":348,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (94 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":489,"context_line":""},{"line_number":490,"context_line":"    def test_cleanup_one_deleting_backup(self):"},{"line_number":491,"context_line":"        \"\"\"Test cleanup_one_backup for backup status \u0027deleting\u0027.\"\"\""},{"line_number":492,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":493,"context_line":""},{"line_number":494,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":495,"context_line":"            status\u003dfields.BackupStatus.DELETING)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_099c5e0d","line":492,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":503,"context_line":""},{"line_number":504,"context_line":"    def test_cleanup_one_deleting_encrypted_backup(self):"},{"line_number":505,"context_line":"        \"\"\"Test cleanup of backup status \u0027deleting\u0027 (encrypted).\"\"\""},{"line_number":506,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":507,"context_line":""},{"line_number":508,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":509,"context_line":"            status\u003dfields.BackupStatus.DELETING,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_699d2209","line":506,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":913,"context_line":"                                              mock_get_backup_device,"},{"line_number":914,"context_line":"                                              mock_get_conn):"},{"line_number":915,"context_line":"        \"\"\"Test backup in-use volume using temp snapshot.\"\"\""},{"line_number":916,"context_line":"        self.override_config(\u0027backup_use_same_host\u0027, True, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":917,"context_line":"        vol_size \u003d 1"},{"line_number":918,"context_line":"        vol_id \u003d self._create_volume_db_entry(size\u003dvol_size,"},{"line_number":919,"context_line":"                                              previous_status\u003d\u0027in-use\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_49a2e64c","line":916,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: E501 line too long (98 \u003e 79 characters)","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":1855,"context_line":"        tpool._nthreads \u003d 20"},{"line_number":1856,"context_line":"        self.assertListEqual([], tpool._threads)"},{"line_number":1857,"context_line":""},{"line_number":1858,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1859,"context_line":""},{"line_number":1860,"context_line":"        self.assertEqual(60, tpool._nthreads)"},{"line_number":1861,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_a98a8abb","line":1858,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"602970dd82b3293402e7a62da488dfec8b0c0fbf","unresolved":false,"context_lines":[{"line_number":1867,"context_line":""},{"line_number":1868,"context_line":"        self.override_config(\u0027backup_native_threads_pool_size\u0027, 100,"},{"line_number":1869,"context_line":"                             configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":1870,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1871,"context_line":""},{"line_number":1872,"context_line":"        self.assertEqual(100, tpool._nthreads)"},{"line_number":1873,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_8987ceb5","line":1870,"updated":"2020-04-27 20:06:58.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"3aabc325b04ffe3f0520b8a657adbb4d5e7c23e5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_e8a85fda","line":62,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E225 missing whitespace around operator","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_48b87329","line":62,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (109 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":239,"context_line":"        def get_admin_context():"},{"line_number":240,"context_line":"            return self.ctxt"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":243,"context_line":"        self.override_config(\u0027periodic_interval\u0027, 0)"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"        vol1_id \u003d self._create_volume_db_entry()"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_28b3674f","line":242,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":345,"context_line":"        # previously used CONF.periodic_interval; see Bug #1828748"},{"line_number":346,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":349,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":350,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":351,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_88839b56","line":348,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (98 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":346,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":349,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":350,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":351,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"},{"line_number":352,"context_line":"        mock_FILC.return_value \u003d mock_init_loop"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_68860f68","line":349,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (94 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":490,"context_line":""},{"line_number":491,"context_line":"    def test_cleanup_one_deleting_backup(self):"},{"line_number":492,"context_line":"        \"\"\"Test cleanup_one_backup for backup status \u0027deleting\u0027.\"\"\""},{"line_number":493,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":494,"context_line":""},{"line_number":495,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":496,"context_line":"            status\u003dfields.BackupStatus.DELETING)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_c87da36f","line":493,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":504,"context_line":""},{"line_number":505,"context_line":"    def test_cleanup_one_deleting_encrypted_backup(self):"},{"line_number":506,"context_line":"        \"\"\"Test cleanup of backup status \u0027deleting\u0027 (encrypted).\"\"\""},{"line_number":507,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":508,"context_line":""},{"line_number":509,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":510,"context_line":"            status\u003dfields.BackupStatus.DELETING,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_a8801748","line":507,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":1284,"context_line":"                                       os_name,"},{"line_number":1285,"context_line":"                                       exp_open_mode):"},{"line_number":1286,"context_line":"        \"\"\"Test normal backup restoration.\"\"\""},{"line_number":1287,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d \u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":1288,"context_line":"        vol_size \u003d 1"},{"line_number":1289,"context_line":"        vol_id \u003d self._create_volume_db_entry("},{"line_number":1290,"context_line":"            status\u003d\u0027restoring-backup\u0027, size\u003dvol_size)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_08978b92","line":1287,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (111 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":1464,"context_line":"        key ID is saved in the backup DB. The backup encryption key ID is"},{"line_number":1465,"context_line":"        gleaned from the restored volume."},{"line_number":1466,"context_line":"        \"\"\""},{"line_number":1467,"context_line":"        # self.backup_mgr.driver_name \u003d \u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":1468,"context_line":"        vol_id \u003d self._create_volume_db_entry(status\u003d\u0027restoring-backup\u0027,"},{"line_number":1469,"context_line":"                                              encryption_key_id\u003dfake.UUID1)"},{"line_number":1470,"context_line":"        backup \u003d self._create_backup_db_entry("}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_e8917f9c","line":1467,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (97 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":1828,"context_line":"        # self.override_config(\u0027backup_driver\u0027,"},{"line_number":1829,"context_line":"        #                      \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027)"},{"line_number":1830,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1831,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d  \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027"},{"line_number":1832,"context_line":"        result \u003d self.backup_mgr.check_support_to_force_delete(self.ctxt)"},{"line_number":1833,"context_line":"        self.assertFalse(result)"},{"line_number":1834,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_48919399","line":1831,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E222 multiple spaces after operator","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":1828,"context_line":"        # self.override_config(\u0027backup_driver\u0027,"},{"line_number":1829,"context_line":"        #                      \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027)"},{"line_number":1830,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1831,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d  \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027"},{"line_number":1832,"context_line":"        result \u003d self.backup_mgr.check_support_to_force_delete(self.ctxt)"},{"line_number":1833,"context_line":"        self.assertFalse(result)"},{"line_number":1834,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_288c077e","line":1831,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: E501 line too long (100 \u003e 79 characters)","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":1861,"context_line":"        tpool._nthreads \u003d 20"},{"line_number":1862,"context_line":"        self.assertListEqual([], tpool._threads)"},{"line_number":1863,"context_line":""},{"line_number":1864,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1865,"context_line":""},{"line_number":1866,"context_line":"        self.assertEqual(60, tpool._nthreads)"},{"line_number":1867,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_88e8bb16","line":1864,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"9f262fc67d8bc9f9bdd4bb7793a7d4ee701792f1","unresolved":false,"context_lines":[{"line_number":1873,"context_line":""},{"line_number":1874,"context_line":"        self.override_config(\u0027backup_native_threads_pool_size\u0027, 100,"},{"line_number":1875,"context_line":"                             configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":1876,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1877,"context_line":""},{"line_number":1878,"context_line":"        self.assertEqual(100, tpool._nthreads)"},{"line_number":1879,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f493fa4_68ebaf11","line":1876,"updated":"2020-05-06 14:14:46.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"0e508c42d57426ab7812616c209f0826998eeb4f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_aa6df2cd","line":62,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E225 missing whitespace around operator","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_8a72aeac","line":62,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (109 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        def get_admin_context():"},{"line_number":241,"context_line":"            return self.ctxt"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":244,"context_line":"        self.override_config(\u0027periodic_interval\u0027, 0)"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"        vol1_id \u003d self._create_volume_db_entry()"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_ea66caee","line":243,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":346,"context_line":"        # previously used CONF.periodic_interval; see Bug #1828748"},{"line_number":347,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":350,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":351,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":352,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_ca6306dd","line":349,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (98 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":347,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":350,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":351,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":352,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"},{"line_number":353,"context_line":"        mock_FILC.return_value \u003d mock_init_loop"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_2a61e2d4","line":350,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (94 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":491,"context_line":""},{"line_number":492,"context_line":"    def test_cleanup_one_deleting_backup(self):"},{"line_number":493,"context_line":"        \"\"\"Test cleanup_one_backup for backup status \u0027deleting\u0027.\"\"\""},{"line_number":494,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":495,"context_line":""},{"line_number":496,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":497,"context_line":"            status\u003dfields.BackupStatus.DELETING)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_0a5e9e11","line":494,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"    def test_cleanup_one_deleting_encrypted_backup(self):"},{"line_number":507,"context_line":"        \"\"\"Test cleanup of backup status \u0027deleting\u0027 (encrypted).\"\"\""},{"line_number":508,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":511,"context_line":"            status\u003dfields.BackupStatus.DELETING,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_6a98facb","line":508,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":1285,"context_line":"                                       os_name,"},{"line_number":1286,"context_line":"                                       exp_open_mode):"},{"line_number":1287,"context_line":"        \"\"\"Test normal backup restoration.\"\"\""},{"line_number":1288,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d \u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":1289,"context_line":"        vol_size \u003d 1"},{"line_number":1290,"context_line":"        vol_id \u003d self._create_volume_db_entry("},{"line_number":1291,"context_line":"            status\u003d\u0027restoring-backup\u0027, size\u003dvol_size)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_4a9d36ba","line":1288,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (111 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":1465,"context_line":"        key ID is saved in the backup DB. The backup encryption key ID is"},{"line_number":1466,"context_line":"        gleaned from the restored volume."},{"line_number":1467,"context_line":"        \"\"\""},{"line_number":1468,"context_line":"        # self.backup_mgr.driver_name \u003d \u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":1469,"context_line":"        vol_id \u003d self._create_volume_db_entry(status\u003d\u0027restoring-backup\u0027,"},{"line_number":1470,"context_line":"                                              encryption_key_id\u003dfake.UUID1)"},{"line_number":1471,"context_line":"        backup \u003d self._create_backup_db_entry("}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_aaa212fb","line":1468,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (97 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":1829,"context_line":"        # self.override_config(\u0027backup_driver\u0027,"},{"line_number":1830,"context_line":"        #                      \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027)"},{"line_number":1831,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1832,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d  \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027"},{"line_number":1833,"context_line":"        result \u003d self.backup_mgr.check_support_to_force_delete(self.ctxt)"},{"line_number":1834,"context_line":"        self.assertFalse(result)"},{"line_number":1835,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_8aa74e0c","line":1832,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E222 multiple spaces after operator","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":1829,"context_line":"        # self.override_config(\u0027backup_driver\u0027,"},{"line_number":1830,"context_line":"        #                      \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027)"},{"line_number":1831,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1832,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d  \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027"},{"line_number":1833,"context_line":"        result \u003d self.backup_mgr.check_support_to_force_delete(self.ctxt)"},{"line_number":1834,"context_line":"        self.assertFalse(result)"},{"line_number":1835,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_eaab6ad4","line":1832,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: E501 line too long (100 \u003e 79 characters)","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":1862,"context_line":"        tpool._nthreads \u003d 20"},{"line_number":1863,"context_line":"        self.assertListEqual([], tpool._threads)"},{"line_number":1864,"context_line":""},{"line_number":1865,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1866,"context_line":""},{"line_number":1867,"context_line":"        self.assertEqual(60, tpool._nthreads)"},{"line_number":1868,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_caa826db","line":1865,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"acf83f31d2bb5ab98d646187fb5a5839034b6d46","unresolved":false,"context_lines":[{"line_number":1874,"context_line":""},{"line_number":1875,"context_line":"        self.override_config(\u0027backup_native_threads_pool_size\u0027, 100,"},{"line_number":1876,"context_line":"                             configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":1877,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1878,"context_line":""},{"line_number":1879,"context_line":"        self.assertEqual(100, tpool._nthreads)"},{"line_number":1880,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_2ab60240","line":1877,"updated":"2020-08-28 15:09:53.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"274996c3e0db0cf7400a67d0d1e7691ed76a7f21"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_5902520a","line":62,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E225 missing whitespace around operator","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":60,"context_line":"        self.backup_mgr.host \u003d \u0027testhost\u0027"},{"line_number":61,"context_line":"        self.backup_mgr.is_initialized \u003d True"},{"line_number":62,"context_line":"        self.backup_mgr.configuration.backup_driver\u003d\u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":63,"context_line":"        self.ctxt \u003d context.get_admin_context()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        paths \u003d [\u0027cinder.volume.rpcapi.VolumeAPI.delete_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_b9056e12","line":62,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (109 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        def get_admin_context():"},{"line_number":241,"context_line":"            return self.ctxt"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":244,"context_line":"        self.override_config(\u0027periodic_interval\u0027, 0)"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"        vol1_id \u003d self._create_volume_db_entry()"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_99086a27","line":243,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":346,"context_line":"        # previously used CONF.periodic_interval; see Bug #1828748"},{"line_number":347,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":350,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":351,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":352,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_f9124657","line":349,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (98 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":347,"context_line":"        new_cfg_name \u003d \u0027backup_driver_init_check_interval\u0027"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.addCleanup(CONF.clear_override, new_cfg_name, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":350,"context_line":"        CONF.set_override(new_cfg_name, new_cfg_value, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":351,"context_line":"        mock_init_loop \u003d mock.MagicMock()"},{"line_number":352,"context_line":"        mock_init_loop.start.side_effect \u003d loopingcall.LoopingCallDone()"},{"line_number":353,"context_line":"        mock_FILC.return_value \u003d mock_init_loop"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_d90dc235","line":350,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (94 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":491,"context_line":""},{"line_number":492,"context_line":"    def test_cleanup_one_deleting_backup(self):"},{"line_number":493,"context_line":"        \"\"\"Test cleanup_one_backup for backup status \u0027deleting\u0027.\"\"\""},{"line_number":494,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":495,"context_line":""},{"line_number":496,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":497,"context_line":"            status\u003dfields.BackupStatus.DELETING)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_39195e79","line":494,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"    def test_cleanup_one_deleting_encrypted_backup(self):"},{"line_number":507,"context_line":"        \"\"\"Test cleanup of backup status \u0027deleting\u0027 (encrypted).\"\"\""},{"line_number":508,"context_line":"        self.override_config(\u0027backup_service_inithost_offload\u0027, False, configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"        backup \u003d self._create_backup_db_entry("},{"line_number":511,"context_line":"            status\u003dfields.BackupStatus.DELETING,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_19145a40","line":508,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (110 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":1285,"context_line":"                                       os_name,"},{"line_number":1286,"context_line":"                                       exp_open_mode):"},{"line_number":1287,"context_line":"        \"\"\"Test normal backup restoration.\"\"\""},{"line_number":1288,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d \u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":1289,"context_line":"        vol_size \u003d 1"},{"line_number":1290,"context_line":"        vol_id \u003d self._create_volume_db_entry("},{"line_number":1291,"context_line":"            status\u003d\u0027restoring-backup\u0027, size\u003dvol_size)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_7928f684","line":1288,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (111 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":1465,"context_line":"        key ID is saved in the backup DB. The backup encryption key ID is"},{"line_number":1466,"context_line":"        gleaned from the restored volume."},{"line_number":1467,"context_line":"        \"\"\""},{"line_number":1468,"context_line":"        # self.backup_mgr.driver_name \u003d \u0027cinder.tests.unit.backup.fake_service.FakeBackupService\u0027"},{"line_number":1469,"context_line":"        vol_id \u003d self._create_volume_db_entry(status\u003d\u0027restoring-backup\u0027,"},{"line_number":1470,"context_line":"                                              encryption_key_id\u003dfake.UUID1)"},{"line_number":1471,"context_line":"        backup \u003d self._create_backup_db_entry("}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_592b7280","line":1468,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (97 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":1829,"context_line":"        # self.override_config(\u0027backup_driver\u0027,"},{"line_number":1830,"context_line":"        #                      \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027)"},{"line_number":1831,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1832,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d  \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027"},{"line_number":1833,"context_line":"        result \u003d self.backup_mgr.check_support_to_force_delete(self.ctxt)"},{"line_number":1834,"context_line":"        self.assertFalse(result)"},{"line_number":1835,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_b91e0e5c","line":1832,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E222 multiple spaces after operator","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":1829,"context_line":"        # self.override_config(\u0027backup_driver\u0027,"},{"line_number":1830,"context_line":"        #                      \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027)"},{"line_number":1831,"context_line":"        self.backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1832,"context_line":"        self.backup_mgr.configuration.backup_driver \u003d  \u0027cinder.backup.drivers.ceph.CephBackupDriver\u0027"},{"line_number":1833,"context_line":"        result \u003d self.backup_mgr.check_support_to_force_delete(self.ctxt)"},{"line_number":1834,"context_line":"        self.assertFalse(result)"},{"line_number":1835,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_99218aa1","line":1832,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: E501 line too long (100 \u003e 79 characters)","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":1862,"context_line":"        tpool._nthreads \u003d 20"},{"line_number":1863,"context_line":"        self.assertListEqual([], tpool._threads)"},{"line_number":1864,"context_line":""},{"line_number":1865,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1866,"context_line":""},{"line_number":1867,"context_line":"        self.assertEqual(60, tpool._nthreads)"},{"line_number":1868,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_f93b66cd","line":1865,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0314d5c0eeacfb8fde38774e69cd51dfd0505851","unresolved":false,"context_lines":[{"line_number":1874,"context_line":""},{"line_number":1875,"context_line":"        self.override_config(\u0027backup_native_threads_pool_size\u0027, 100,"},{"line_number":1876,"context_line":"                             configuration.SHARED_BACKUP_CONF_GROUP)"},{"line_number":1877,"context_line":"        backup_mgr \u003d importutils.import_object(CONF.backup_manager)"},{"line_number":1878,"context_line":""},{"line_number":1879,"context_line":"        self.assertEqual(100, tpool._nthreads)"},{"line_number":1880,"context_line":"        self.assertListEqual([], tpool._threads)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_d93662e7","line":1877,"updated":"2020-09-11 18:06:34.000000000","message":"pep8: F841 local variable \u0027backup_mgr\u0027 is assigned to but never used","commit_id":"7e472abe13b6fbcab4e87434edf1345a1ff561c9"}],"cinder/volume/configuration.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0f14154d9f6cbd7b85bd9c41e37f46d40127335c","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        except cfg.NoSuchOptError:"},{"line_number":128,"context_line":"            return None"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def __getattr__(self, opt_name):"},{"line_number":131,"context_line":"        # Don\u0027t use self.X to avoid reentrant call to __getattr__()"},{"line_number":132,"context_line":"        backend_conf \u003d object.__getattribute__(self, \u0027backend_conf\u0027)"},{"line_number":133,"context_line":"        opt_value \u003d getattr(backend_conf, opt_name)"},{"line_number":134,"context_line":"        if opt_value is None:"},{"line_number":135,"context_line":"            shared_conf \u003d object.__getattribute__(self, \u0027shared_backend_conf\u0027)"},{"line_number":136,"context_line":"            opt_value \u003d getattr(shared_conf, opt_name)"},{"line_number":137,"context_line":"        return opt_value"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"class BackupGroupConfiguration(BackendGroupConfiguration):"}],"source_content_type":"text/x-python","patch_set":11,"id":"9f560f44_246dd698","line":137,"range":{"start_line":130,"start_character":0,"end_line":137,"end_character":24},"updated":"2020-10-12 19:23:43.000000000","message":"I think you may need to override this function in your BackupGroupConfiguration class, note the hard-coded values in lines 132 and 135","commit_id":"c2e173ec7109ddcc405aba7bb13d015a31fdd603"}]}