)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"a42d79fdf88e692e07f8b84fd8e4472d815e7836","unresolved":false,"context_lines":[{"line_number":12,"context_line":"the function object obj_to_primitive instead of serializing the result"},{"line_number":13,"context_line":"of the call of obj_to_primitive. This caused a false positive test"},{"line_number":14,"context_line":"covering the error in the implementation as well."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ef4a23aa2bf5cb46b481045f3d968f62f74606d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bfdaf3ff_1f0a6189","line":15,"updated":"2019-01-16 23:23:22.000000000","message":"no bug for this?","commit_id":"d602a7737fa96b4dfaede871df87c86c0aef7bd5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"39bb4e21a07f687d50072ef454e194aaeab94ebf","unresolved":false,"context_lines":[{"line_number":12,"context_line":"the function object obj_to_primitive instead of serializing the result"},{"line_number":13,"context_line":"of the call of obj_to_primitive. This caused a false positive test"},{"line_number":14,"context_line":"covering the error in the implementation as well."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ef4a23aa2bf5cb46b481045f3d968f62f74606d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bfdaf3ff_b3b589cf","line":15,"in_reply_to":"bfdaf3ff_1f0a6189","updated":"2019-01-17 08:18:44.000000000","message":"filed a bug","commit_id":"d602a7737fa96b4dfaede871df87c86c0aef7bd5"}],"nova/objects/build_request.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"a42d79fdf88e692e07f8b84fd8e4472d815e7836","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                      \u0027with uuid %(instance_uuid)s. Found version %(version)s \u0027"},{"line_number":81,"context_line":"                      \u0027which is not supported here.\u0027,"},{"line_number":82,"context_line":"                      dict(instance_uuid\u003dself.instance_uuid,"},{"line_number":83,"context_line":"                           version\u003djsonutils.loads("},{"line_number":84,"context_line":"                               db_instance)[\"nova_object.version\"]))"},{"line_number":85,"context_line":"            LOG.exception(\u0027Could not deserialize instance in BuildRequest\u0027)"},{"line_number":86,"context_line":"            raise exception.BuildRequestNotFound(uuid\u003dself.instance_uuid)"},{"line_number":87,"context_line":"        # NOTE(sbauza): The instance primitive should already have the deleted"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfdaf3ff_3fac0533","line":84,"range":{"start_line":83,"start_character":35,"end_line":84,"end_character":66},"updated":"2019-01-16 23:23:22.000000000","message":"it looks like the original intent was to grab the objver kwarg that was passed in for the message associated with IncompatibleObjectVersion, which would be exc.kwargs[\u0027objver\u0027]. That said, I\u0027m not sure we can (or should) count on the exception being an actual IncompatibleObjectVersion (as opposed to a subclass, or having its message overridden); so what you\u0027ve done here is safer, if pretty heavy.","commit_id":"d602a7737fa96b4dfaede871df87c86c0aef7bd5"}]}
