)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"2cdaf1c14820c85014f177e87a78404d88d4e1c2","unresolved":false,"context_lines":[{"line_number":1170,"context_line":""},{"line_number":1171,"context_line":"        try:"},{"line_number":1172,"context_line":"            # checking that instance was not already evacuated to other host"},{"line_number":1173,"context_line":"            if self.driver.capabilities[\"supports_evacuate\"]:"},{"line_number":1174,"context_line":"                self._destroy_evacuated_instances(context)"},{"line_number":1175,"context_line":"            for instance in instances:"},{"line_number":1176,"context_line":"                self._init_instance(context, instance)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_89d45548","line":1173,"updated":"2015-03-05 11:32:53.000000000","message":"I think the name \u0027supports_evacuate\u0027\nis really confusing\ncan make someone think the hypervisor support evacuation action itself","commit_id":"601864e79e71b069b7328f80b3fbbd97c53c4c0f"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"69ef5dff3afd0ade02a62b4c38c540795a2932b0","unresolved":false,"context_lines":[{"line_number":1170,"context_line":""},{"line_number":1171,"context_line":"        try:"},{"line_number":1172,"context_line":"            # checking that instance was not already evacuated to other host"},{"line_number":1173,"context_line":"            if self.driver.capabilities[\"supports_evacuate\"]:"},{"line_number":1174,"context_line":"                self._destroy_evacuated_instances(context)"},{"line_number":1175,"context_line":"            for instance in instances:"},{"line_number":1176,"context_line":"                self._init_instance(context, instance)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_a93cb9e3","line":1173,"in_reply_to":"9a80dd14_89d45548","updated":"2015-03-05 11:49:49.000000000","message":"i think that it is under the heading driver capabilities. does that make sense?\nthis is actually a driver flag and not a hypervisor one","commit_id":"601864e79e71b069b7328f80b3fbbd97c53c4c0f"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"42d80e36dbd223a967b03854b0247fc2ad79e3e4","unresolved":false,"context_lines":[{"line_number":1170,"context_line":""},{"line_number":1171,"context_line":"        try:"},{"line_number":1172,"context_line":"            # checking that instance was not already evacuated to other host"},{"line_number":1173,"context_line":"            if self.driver.capabilities[\"supports_evacuate\"]:"},{"line_number":1174,"context_line":"                self._destroy_evacuated_instances(context)"},{"line_number":1175,"context_line":"            for instance in instances:"},{"line_number":1176,"context_line":"                self._init_instance(context, instance)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_24c5503c","line":1173,"in_reply_to":"9a80dd14_a93cb9e3","updated":"2015-03-05 11:58:15.000000000","message":"but actually it\u0027s not driver support evacuate \nit\u0027s we will not destroy the evacuated instances, \nif we are going to disable whole evacuate action \nI think the name is proper..","commit_id":"601864e79e71b069b7328f80b3fbbd97c53c4c0f"}]}
