)]}'
{"nova/compute/api.py":[{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"01e054a0faf5af2174661c77878e3ae042d49df0","unresolved":false,"context_lines":[{"line_number":2209,"context_line":"    #                intended to prevent accidental change/delete of instances"},{"line_number":2210,"context_line":"    @wrap_check_policy"},{"line_number":2211,"context_line":"    @check_instance_cell"},{"line_number":2212,"context_line":"    @check_instance_state(vm_state\u003d[vm_states.ACTIVE, vm_states.STOPPED,"},{"line_number":2213,"context_line":"                                    vm_states.PAUSED, vm_states.SUSPENDED])"},{"line_number":2214,"context_line":"    def snapshot(self, context, instance, name, extra_properties\u003dNone):"},{"line_number":2215,"context_line":"        \"\"\"Snapshot the given instance."}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_dc928257","line":2212,"updated":"2015-09-25 04:29:08.000000000","message":"Should we also add PAUSED state?","commit_id":"4481567f1f1254b9a74345261a84f415293a1a61"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"72d3d3163e60fb74d1a07b3b86c5be3606c434b4","unresolved":false,"context_lines":[{"line_number":2209,"context_line":"    #                intended to prevent accidental change/delete of instances"},{"line_number":2210,"context_line":"    @wrap_check_policy"},{"line_number":2211,"context_line":"    @check_instance_cell"},{"line_number":2212,"context_line":"    @check_instance_state(vm_state\u003d[vm_states.ACTIVE, vm_states.STOPPED,"},{"line_number":2213,"context_line":"                                    vm_states.PAUSED, vm_states.SUSPENDED])"},{"line_number":2214,"context_line":"    def snapshot(self, context, instance, name, extra_properties\u003dNone):"},{"line_number":2215,"context_line":"        \"\"\"Snapshot the given instance."}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_13c8ad21","line":2212,"in_reply_to":"9a1a9d01_dc928257","updated":"2015-09-28 07:47:33.000000000","message":"Actually I think we shouldn\u0027t support PAUSED state when we do snapshot, because paused instance will be converted to suspended state in current libvirt driver implement. The internal state converting proxy looks no make sense. The trick is implemented in driver, it can\u0027t impact the instance that boot from volume. \n\nhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1397-1401","commit_id":"4481567f1f1254b9a74345261a84f415293a1a61"}]}