)]}'
{"nova/conductor/manager.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"1b82482161dc98a65e1942084d9669b0704edcc4","unresolved":false,"context_lines":[{"line_number":966,"context_line":"                                 \u0027task_state\u0027: None}, ex, request_spec)"},{"line_number":967,"context_line":"                        LOG.warning(\u0027Rebuild failed: %s\u0027,"},{"line_number":968,"context_line":"                                    six.text_type(ex), instance\u003dinstance)"},{"line_number":969,"context_line":"                        compute_utils.add_instance_fault_from_exc(context,"},{"line_number":970,"context_line":"                            instance, ex, sys.exc_info())"},{"line_number":971,"context_line":""},{"line_number":972,"context_line":"            compute_utils.notify_about_instance_usage("},{"line_number":973,"context_line":"                self.notifier, context, instance, \"rebuild.scheduled\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f93b717_33ad0988","line":970,"range":{"start_line":969,"start_character":24,"end_line":970,"end_character":57},"updated":"2018-01-29 19:15:06.000000000","message":"Hm, after looking at set_vm_state_and_notify I realize I think this is redundant. It\u0027s not reflected in the unit tests because all of them mock out set_vm_state_and_notify.\n\nMaybe we should remove this and unmock set_vm_state_and_notify if we can.","commit_id":"bccf26c93a973d000e4339843ce9256814286d10"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"11fa81b3561464c6b39ab6e30e7d0ce0f6daed65","unresolved":false,"context_lines":[{"line_number":966,"context_line":"                                 \u0027task_state\u0027: None}, ex, request_spec)"},{"line_number":967,"context_line":"                        LOG.warning(\u0027Rebuild failed: %s\u0027,"},{"line_number":968,"context_line":"                                    six.text_type(ex), instance\u003dinstance)"},{"line_number":969,"context_line":"                        compute_utils.add_instance_fault_from_exc(context,"},{"line_number":970,"context_line":"                            instance, ex, sys.exc_info())"},{"line_number":971,"context_line":""},{"line_number":972,"context_line":"            compute_utils.notify_about_instance_usage("},{"line_number":973,"context_line":"                self.notifier, context, instance, \"rebuild.scheduled\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f93b717_2692a59f","line":970,"range":{"start_line":969,"start_character":24,"end_line":970,"end_character":57},"in_reply_to":"5f93b717_33ad0988","updated":"2018-01-29 19:38:45.000000000","message":"Yeah now that I did the backports, I see that was added in https://review.openstack.org/#/c/536268/. I\u0027ll push another patch on top to remove that.","commit_id":"bccf26c93a973d000e4339843ce9256814286d10"}]}