)]}'
{"nova/compute/api.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8b3d19602235d73ca6a3e93895b1a02f69d1bfb8","unresolved":false,"context_lines":[{"line_number":3943,"context_line":""},{"line_number":3944,"context_line":"    # TODO(stephenfin): This logic would be so much easier to grok if we"},{"line_number":3945,"context_line":"    # finally split resize and cold migration into separate code paths"},{"line_number":3946,"context_line":"    # TODO(stephenfin): The \u0027block_accelerators\u0027 decorator doesn\u0027t take into"},{"line_number":3947,"context_line":"    # account the accelerators requested in the new flavor"},{"line_number":3948,"context_line":"    @block_accelerators()"},{"line_number":3949,"context_line":"    @check_instance_lock"},{"line_number":3950,"context_line":"    @check_instance_state(vm_state\u003d[vm_states.ACTIVE, vm_states.STOPPED])"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_47451a15","line":3947,"range":{"start_line":3946,"start_character":0,"end_line":3947,"end_character":58},"updated":"2020-08-31 10:16:07.000000000","message":"You forgot to remove this","commit_id":"7bcbe9a6ed2d5123a6f2e01e010f9186a1875ba5"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"da3d06153b5b9adeb45c5aac17a576fe663f5e28","unresolved":false,"context_lines":[{"line_number":3943,"context_line":""},{"line_number":3944,"context_line":"    # TODO(stephenfin): This logic would be so much easier to grok if we"},{"line_number":3945,"context_line":"    # finally split resize and cold migration into separate code paths"},{"line_number":3946,"context_line":"    # TODO(stephenfin): The \u0027block_accelerators\u0027 decorator doesn\u0027t take into"},{"line_number":3947,"context_line":"    # account the accelerators requested in the new flavor"},{"line_number":3948,"context_line":"    @block_accelerators()"},{"line_number":3949,"context_line":"    @check_instance_lock"},{"line_number":3950,"context_line":"    @check_instance_state(vm_state\u003d[vm_states.ACTIVE, vm_states.STOPPED])"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_0b7cbfe9","line":3947,"range":{"start_line":3946,"start_character":0,"end_line":3947,"end_character":58},"in_reply_to":"9f560f44_47451a15","updated":"2020-09-01 05:44:04.000000000","message":"Done","commit_id":"7bcbe9a6ed2d5123a6f2e01e010f9186a1875ba5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8b3d19602235d73ca6a3e93895b1a02f69d1bfb8","unresolved":false,"context_lines":[{"line_number":3980,"context_line":"        else:"},{"line_number":3981,"context_line":"            new_instance_type \u003d flavors.get_flavor_by_flavor_id("},{"line_number":3982,"context_line":"                    flavor_id, read_deleted\u003d\"no\")"},{"line_number":3983,"context_line":"            if new_instance_type.extra_specs.get(\u0027accel:device_profile\u0027):"},{"line_number":3984,"context_line":"                raise exception.ForbiddenWithAccelerators()"},{"line_number":3985,"context_line":"            # Check to see if we\u0027re resizing to a zero-disk flavor which is"},{"line_number":3986,"context_line":"            # only supported with volume-backed servers."}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_c7392aa4","line":3983,"updated":"2020-08-31 10:16:07.000000000","message":"A comment on this would be helpful. The casual reader would see the decorator and wonder why this is necessary.","commit_id":"7bcbe9a6ed2d5123a6f2e01e010f9186a1875ba5"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"da3d06153b5b9adeb45c5aac17a576fe663f5e28","unresolved":false,"context_lines":[{"line_number":3980,"context_line":"        else:"},{"line_number":3981,"context_line":"            new_instance_type \u003d flavors.get_flavor_by_flavor_id("},{"line_number":3982,"context_line":"                    flavor_id, read_deleted\u003d\"no\")"},{"line_number":3983,"context_line":"            if new_instance_type.extra_specs.get(\u0027accel:device_profile\u0027):"},{"line_number":3984,"context_line":"                raise exception.ForbiddenWithAccelerators()"},{"line_number":3985,"context_line":"            # Check to see if we\u0027re resizing to a zero-disk flavor which is"},{"line_number":3986,"context_line":"            # only supported with volume-backed servers."}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_2b77030b","line":3983,"in_reply_to":"9f560f44_c7392aa4","updated":"2020-09-01 05:44:04.000000000","message":"Done","commit_id":"7bcbe9a6ed2d5123a6f2e01e010f9186a1875ba5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8ab70ce7f4004874b748438ffeb005e12998163b","unresolved":false,"context_lines":[{"line_number":3979,"context_line":"            new_instance_type \u003d flavors.get_flavor_by_flavor_id("},{"line_number":3980,"context_line":"                    flavor_id, read_deleted\u003d\"no\")"},{"line_number":3981,"context_line":"            # NOTE(wenping): We use this instead of the \u0027block_accelerator\u0027"},{"line_number":3982,"context_line":"            # decorator since the operation can differ depending on args, and for"},{"line_number":3983,"context_line":"            # resize we have two flavors to worry about, we should reject resize"},{"line_number":3984,"context_line":"            # with new flavor with accelerator."},{"line_number":3985,"context_line":"            if new_instance_type.extra_specs.get(\u0027accel:device_profile\u0027):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_353927d8","line":3982,"updated":"2020-08-31 15:36:03.000000000","message":"pep8: E501 line too long (81 \u003e 79 characters)","commit_id":"670fd7167a1316af0ee388c0364935a8ee1df6ed"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8ab70ce7f4004874b748438ffeb005e12998163b","unresolved":false,"context_lines":[{"line_number":3980,"context_line":"                    flavor_id, read_deleted\u003d\"no\")"},{"line_number":3981,"context_line":"            # NOTE(wenping): We use this instead of the \u0027block_accelerator\u0027"},{"line_number":3982,"context_line":"            # decorator since the operation can differ depending on args, and for"},{"line_number":3983,"context_line":"            # resize we have two flavors to worry about, we should reject resize"},{"line_number":3984,"context_line":"            # with new flavor with accelerator."},{"line_number":3985,"context_line":"            if new_instance_type.extra_specs.get(\u0027accel:device_profile\u0027):"},{"line_number":3986,"context_line":"                raise exception.ForbiddenWithAccelerators()"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_1536a3e8","line":3983,"updated":"2020-08-31 15:36:03.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"670fd7167a1316af0ee388c0364935a8ee1df6ed"}]}