)]}'
{"cinder/volume/drivers/dell_emc/powermax/common.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"5f2711a1317cbf4c2f0084ef8b0cbf60a031a598","unresolved":false,"context_lines":[{"line_number":6180,"context_line":"                \u0027recreate the pairs as needed.\u0027) % ("},{"line_number":6181,"context_line":"                rdf_group_no, local_rdfg_device_count, remote_rdf_group_no,"},{"line_number":6182,"context_line":"                remote_rdfg_device_count))"},{"line_number":6183,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":6184,"context_line":""},{"line_number":6185,"context_line":"    def _validate_storage_group_is_replication_enabled("},{"line_number":6186,"context_line":"            self, array, storage_group_name):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_82826251","line":6183,"range":{"start_line":6183,"start_character":28,"end_line":6183,"end_character":53},"updated":"2020-06-10 20:03:46.000000000","message":"VolumeDriverException should be raised here","commit_id":"88999b76db4f0a60b0fb266ff8c6d6ab975aff2d"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"7eda993662016c90620ac45e8c574d850d2b1d90","unresolved":false,"context_lines":[{"line_number":6180,"context_line":"                \u0027recreate the pairs as needed.\u0027) % ("},{"line_number":6181,"context_line":"                rdf_group_no, local_rdfg_device_count, remote_rdf_group_no,"},{"line_number":6182,"context_line":"                remote_rdfg_device_count))"},{"line_number":6183,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":6184,"context_line":""},{"line_number":6185,"context_line":"    def _validate_storage_group_is_replication_enabled("},{"line_number":6186,"context_line":"            self, array, storage_group_name):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_81b363e9","line":6183,"range":{"start_line":6183,"start_character":28,"end_line":6183,"end_character":53},"in_reply_to":"ff570b3c_82826251","updated":"2020-06-12 14:37:18.000000000","message":"Thanks Ivan, we will change this","commit_id":"88999b76db4f0a60b0fb266ff8c6d6ab975aff2d"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"ac4932d48edd286771a1761eef48a2862d1b08b2","unresolved":false,"context_lines":[{"line_number":6177,"context_line":"                \u0027local and remote RDFGs. Local RDFG %s has %s devices. Remote \u0027"},{"line_number":6178,"context_line":"                \u0027RDFG %s has %s devices. The same number of devices is \u0027"},{"line_number":6179,"context_line":"                \u0027expected. Check RDFGs for broken RDF pairs and cleanup or \u0027"},{"line_number":6180,"context_line":"                \u0027recreate the pairs as needed.\u0027) % ("},{"line_number":6181,"context_line":"                rdf_group_no, local_rdfg_device_count, remote_rdf_group_no,"},{"line_number":6182,"context_line":"                remote_rdfg_device_count))"},{"line_number":6183,"context_line":"            raise exception.VolumeDriverException(msg)"},{"line_number":6184,"context_line":""},{"line_number":6185,"context_line":"    def _validate_storage_group_is_replication_enabled("}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_de7807c5","line":6182,"range":{"start_line":6180,"start_character":51,"end_line":6182,"end_character":42},"updated":"2020-06-28 18:49:01.000000000","message":"String formatting with more than two variables should use named values according to our guidelines. So something like:\n\n\"Local RDFG %(rdfgroup)s has %(localdevcount)s...\" % {\n    \u0027rdfgroup\u0027: rdf_group_no,\n    \u0027localdevcount\u0027: local_rdfg_device_count,\n    ...}","commit_id":"984be92ae962f5c25ba5823ed1143df7acef9df4"}]}