)]}'
{"trove/common/strategies/cluster/experimental/mongodb/api.py":[{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"0e7dc9beac30b2045b69f476c353aab990eca6c8","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                                        [], [], datastore,"},{"line_number":134,"context_line":"                                        datastore_version,"},{"line_number":135,"context_line":"                                        volume_size, None,"},{"line_number":136,"context_line":"                                        availability_zone\u003dazs,"},{"line_number":137,"context_line":"                                        nics\u003dnics,"},{"line_number":138,"context_line":"                                        configuration_id\u003dNone,"},{"line_number":139,"context_line":"                                        cluster_config\u003dmember_config,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_48835405","line":136,"updated":"2018-05-22 14:51:21.000000000","message":"Thanks for the patch. As \u0027availability_zone\u0027 itself defines that it should contain a single value whereas you are passing a whole list. I think it should contain azs[i] only.The value will be updated during the for loop execution.","commit_id":"b11c3fdc5d0db8d86ac74ccbd686d852b4fd8bd3"},{"author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"change_message_id":"eded868c7fe30832e18ba4fedbd1cfff96321c93","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                                        [], [], datastore,"},{"line_number":134,"context_line":"                                        datastore_version,"},{"line_number":135,"context_line":"                                        volume_size, None,"},{"line_number":136,"context_line":"                                        availability_zone\u003dazs,"},{"line_number":137,"context_line":"                                        nics\u003dnics,"},{"line_number":138,"context_line":"                                        configuration_id\u003dNone,"},{"line_number":139,"context_line":"                                        cluster_config\u003dmember_config,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_6067d74e","line":136,"in_reply_to":"5f7c97a3_48835405","updated":"2018-05-31 02:52:57.000000000","message":"tianhui use azs \u003d instances[0].get(\u0027availability_zone\u0027, None)\n So in this context, \u0027azs\u0027 is a string actually.","commit_id":"b11c3fdc5d0db8d86ac74ccbd686d852b4fd8bd3"},{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"0e7dc9beac30b2045b69f476c353aab990eca6c8","unresolved":false,"context_lines":[{"line_number":150,"context_line":"                                        [], [], datastore,"},{"line_number":151,"context_line":"                                        datastore_version,"},{"line_number":152,"context_line":"                                        volume_size, None,"},{"line_number":153,"context_line":"                                        availability_zone\u003dazs,"},{"line_number":154,"context_line":"                                        nics\u003dnics,"},{"line_number":155,"context_line":"                                        configuration_id\u003dNone,"},{"line_number":156,"context_line":"                                        cluster_config\u003dconfigsvr_config,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_68865016","line":153,"updated":"2018-05-22 14:51:21.000000000","message":"same as here. For instance like in region_name","commit_id":"b11c3fdc5d0db8d86ac74ccbd686d852b4fd8bd3"},{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"0e7dc9beac30b2045b69f476c353aab990eca6c8","unresolved":false,"context_lines":[{"line_number":167,"context_line":"                                        [], [], datastore,"},{"line_number":168,"context_line":"                                        datastore_version,"},{"line_number":169,"context_line":"                                        volume_size, None,"},{"line_number":170,"context_line":"                                        availability_zone\u003dazs,"},{"line_number":171,"context_line":"                                        nics\u003dnics,"},{"line_number":172,"context_line":"                                        configuration_id\u003dNone,"},{"line_number":173,"context_line":"                                        cluster_config\u003dmongos_config,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_087d5c1b","line":170,"updated":"2018-05-22 14:51:21.000000000","message":"same","commit_id":"b11c3fdc5d0db8d86ac74ccbd686d852b4fd8bd3"}],"trove/tests/unittests/cluster/test_mongodb_cluster.py":[{"author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"change_message_id":"eded868c7fe30832e18ba4fedbd1cfff96321c93","unresolved":false,"context_lines":[{"line_number":242,"context_line":"                            self.datastore,"},{"line_number":243,"context_line":"                            self.datastore_version,"},{"line_number":244,"context_line":"                            self.instances, {}, None, None)"},{"line_number":245,"context_line":"        azs \u003d \"foo-azs\""},{"line_number":246,"context_line":"        azs_count \u003d [kw.get(\u0027azs\u0027) for _, kw in"},{"line_number":247,"context_line":"                     mock_ins_create.call_args_list].count(azs)"},{"line_number":248,"context_line":"        self.assertEqual(0, azs_count)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_807473ff","line":245,"range":{"start_line":245,"start_character":8,"end_line":245,"end_character":11},"updated":"2018-05-31 02:52:57.000000000","message":"there is no parameters named azs in Instance.create. Your should use availability_zone instead.","commit_id":"b11c3fdc5d0db8d86ac74ccbd686d852b4fd8bd3"}]}