)]}'
{"cinder/volume/flows/api/create_volume.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"268410b15bc48df744b307d8653e438f59fab08b","unresolved":false,"context_lines":[{"line_number":287,"context_line":"        for (az_name, disabled) in az_data:"},{"line_number":288,"context_line":"            tracked_disabled \u003d disabled_map.get(az_name, True)"},{"line_number":289,"context_line":"            disabled_map[az_name] \u003d tracked_disabled and disabled"},{"line_number":290,"context_line":"        raw_zones \u003d [{\u0027name\u0027: name, \u0027available\u0027: not disabled}"},{"line_number":291,"context_line":"                     for (name, disabled) in disabled_map.items()]"},{"line_number":292,"context_line":"        azs \u003d set([az[\u0027name\u0027] for az in raw_zones])"},{"line_number":293,"context_line":"        return availability_zone in azs"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    def _extract_availability_zone(self, availability_zone, snapshot,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff0f0b1f_6bbd77b5","line":292,"range":{"start_line":290,"start_character":7,"end_line":292,"end_character":51},"updated":"2017-05-25 09:28:41.000000000","message":"These two lines can be combined, and it seems that the \u0027available\u0027 attribute is never used.","commit_id":"bf05b6cb782e276783b84f7e512630c6927024cf"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"c1aa09a4ffa9218449473966a9ab6c7fde700496","unresolved":false,"context_lines":[{"line_number":287,"context_line":"        for (az_name, disabled) in az_data:"},{"line_number":288,"context_line":"            tracked_disabled \u003d disabled_map.get(az_name, True)"},{"line_number":289,"context_line":"            disabled_map[az_name] \u003d tracked_disabled and disabled"},{"line_number":290,"context_line":"        raw_zones \u003d [{\u0027name\u0027: name, \u0027available\u0027: not disabled}"},{"line_number":291,"context_line":"                     for (name, disabled) in disabled_map.items()]"},{"line_number":292,"context_line":"        azs \u003d set([az[\u0027name\u0027] for az in raw_zones])"},{"line_number":293,"context_line":"        return availability_zone in azs"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"    def _extract_availability_zone(self, availability_zone, snapshot,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff0f0b1f_e669b924","line":292,"range":{"start_line":290,"start_character":7,"end_line":292,"end_character":51},"in_reply_to":"ff0f0b1f_6bbd77b5","updated":"2017-05-25 11:35:46.000000000","message":"Done","commit_id":"bf05b6cb782e276783b84f7e512630c6927024cf"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"6eab9f845c318c5142c6a48754b71aeacc6cae71","unresolved":false,"context_lines":[{"line_number":279,"context_line":"                       \u0027volume_type\u0027: volume_type})"},{"line_number":280,"context_line":"            return volume_type"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"    def _check_and_update_availability_zone(self, availability_zone):"},{"line_number":283,"context_line":"        topic \u003d constants.VOLUME_TOPIC"},{"line_number":284,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":285,"context_line":"        services \u003d objects.ServiceList.get_all_by_topic(ctxt, topic)"}],"source_content_type":"text/x-python","patch_set":4,"id":"df140735_80b4bc36","line":282,"updated":"2017-05-31 20:53:27.000000000","message":"It makes sense to add unit test only for this method","commit_id":"802799cfae9cc5215c5ef37f3bf287f2c9672c8b"}]}