Commit e4f88809 authored by Georges Racinet's avatar Georges Racinet 🦑
Browse files

Bumped Gitaly protocol to v13.10.3

Nothing of consequence to us:

- more comments
- removal of `name` field for Git remotes (mirrors-related)
  operations
parent 92c2deb0bf56
Pipeline #21012 passed with stages
in 1 minute and 43 seconds
......@@ -60,9 +60,12 @@
raise NotImplementedError('Method not implemented!')
def GetLFSPointers(self, request, context):
"""Missing associated documentation comment in .proto file."""
"""GetLFSPointers retrieves LFS pointers from a given set of object IDs.
This RPC filters all requested objects and only returns those which refer
to a valid LFS pointer.
"""
context.set_code(grpc.StatusCode.UNIMPLEMENTED)
context.set_details('Method not implemented!')
raise NotImplementedError('Method not implemented!')
def GetNewLFSPointers(self, request, context):
......@@ -64,11 +67,14 @@
context.set_code(grpc.StatusCode.UNIMPLEMENTED)
context.set_details('Method not implemented!')
raise NotImplementedError('Method not implemented!')
def GetNewLFSPointers(self, request, context):
"""Missing associated documentation comment in .proto file."""
"""GetNewLFSPointers retrieves LFS pointers for a limited subset of the
commit graph. It will return all LFS pointers which are reachable by the
provided revision, but not reachable by any of the limiting references.
"""
context.set_code(grpc.StatusCode.UNIMPLEMENTED)
context.set_details('Method not implemented!')
raise NotImplementedError('Method not implemented!')
def GetAllLFSPointers(self, request, context):
......@@ -70,9 +76,10 @@
context.set_code(grpc.StatusCode.UNIMPLEMENTED)
context.set_details('Method not implemented!')
raise NotImplementedError('Method not implemented!')
def GetAllLFSPointers(self, request, context):
"""Missing associated documentation comment in .proto file."""
"""GetAllLFSPointers retrieves all LFS pointers of the given repository.
"""
context.set_code(grpc.StatusCode.UNIMPLEMENTED)
context.set_details('Method not implemented!')
raise NotImplementedError('Method not implemented!')
......
This diff is collapsed.
......@@ -462,7 +462,10 @@
raise NotImplementedError('Method not implemented!')
def RemoveRepository(self, request, context):
"""Missing associated documentation comment in .proto file."""
"""RemoveRepository will move the repository to `+gitaly/tmp/<relative_path>_removed` and
eventually remove it. This ensures that even on networked filesystems the
data is actually removed even if there's someone still handling the data.
"""
context.set_code(grpc.StatusCode.UNIMPLEMENTED)
context.set_details('Method not implemented!')
raise NotImplementedError('Method not implemented!')
......
......@@ -21,8 +21,12 @@
op: ACCESSOR
};
}
// GetLFSPointers retrieves LFS pointers from a given set of object IDs.
// This RPC filters all requested objects and only returns those which refer
// to a valid LFS pointer.
rpc GetLFSPointers(GetLFSPointersRequest) returns (stream GetLFSPointersResponse) {
option (op_type) = {
op: ACCESSOR
};
}
......@@ -24,10 +28,14 @@
rpc GetLFSPointers(GetLFSPointersRequest) returns (stream GetLFSPointersResponse) {
option (op_type) = {
op: ACCESSOR
};
}
// GetNewLFSPointers retrieves LFS pointers for a limited subset of the
// commit graph. It will return all LFS pointers which are reachable by the
// provided revision, but not reachable by any of the limiting references.
rpc GetNewLFSPointers(GetNewLFSPointersRequest) returns (stream GetNewLFSPointersResponse) {
option (op_type) = {
op: ACCESSOR
};
}
......@@ -29,8 +37,10 @@
rpc GetNewLFSPointers(GetNewLFSPointersRequest) returns (stream GetNewLFSPointersResponse) {
option (op_type) = {
op: ACCESSOR
};
}
// GetAllLFSPointers retrieves all LFS pointers of the given repository.
rpc GetAllLFSPointers(GetAllLFSPointersRequest) returns (stream GetAllLFSPointersResponse) {
option (op_type) = {
op: ACCESSOR
......@@ -84,4 +94,5 @@
ObjectType type = 8;
}
// LFSPointer is a git blob which points to an LFS object.
message LFSPointer {
......@@ -87,2 +98,4 @@
message LFSPointer {
// Size is the size of the blob. This is not the size of the LFS object
// pointed to.
int64 size = 1;
......@@ -88,2 +101,4 @@
int64 size = 1;
// Data is the bare data of the LFS pointer blob. It contains the pointer to
// the LFS data in the format specified by the LFS project.
bytes data = 2;
......@@ -89,4 +104,5 @@
bytes data = 2;
// Oid is the object ID of the blob.
string oid = 3;
}
......@@ -96,4 +112,5 @@
bytes path = 3;
}
// GetLFSPointersRequest is a request for the GetLFSPointers RPC.
message GetLFSPointersRequest {
......@@ -99,3 +116,4 @@
message GetLFSPointersRequest {
// Repository is the repository for which LFS pointers should be retrieved
// from.
Repository repository = 1[(target_repository)=true];
......@@ -101,4 +119,6 @@
Repository repository = 1[(target_repository)=true];
// BlobIds is the list of blobs to retrieve LFS pointers from. Must be a
// non-empty list of blobs IDs to fetch.
repeated string blob_ids = 2;
}
......@@ -102,4 +122,5 @@
repeated string blob_ids = 2;
}
// GetLFSPointersResponse is a response for the GetLFSPointers RPC.
message GetLFSPointersResponse {
......@@ -105,4 +126,5 @@
message GetLFSPointersResponse {
// LfsPointers is the list of LFS pointers which were requested.
repeated LFSPointer lfs_pointers = 1;
}
......@@ -106,4 +128,5 @@
repeated LFSPointer lfs_pointers = 1;
}
// GetNewLFSPointersRequest is a request for the GetNewLFSPointers RPC.
message GetNewLFSPointersRequest {
......@@ -109,3 +132,4 @@
message GetNewLFSPointersRequest {
// Repository is the repository for which LFS pointers should be retrieved
// from.
Repository repository = 1[(target_repository)=true];
......@@ -111,2 +135,3 @@
Repository repository = 1[(target_repository)=true];
// Revision is the revision for which to retrieve new LFS pointers.
bytes revision = 2;
......@@ -112,2 +137,3 @@
bytes revision = 2;
// Limit limits the number of LFS pointers returned.
int32 limit = 3;
......@@ -113,3 +139,5 @@
int32 limit = 3;
// Note: When `not_in_all` is true, `not_in_refs` is ignored
// NotInAll limits the revision graph to not include any commits which are
// referenced by a git reference. When `not_in_all` is true, `not_in_refs` is
// ignored.
bool not_in_all = 4;
......@@ -115,4 +143,7 @@
bool not_in_all = 4;
// NotInRefs is a list of references used to limit the revision graph. Any
// commit reachable by any commit in NotInRefs will not be searched for new
// LFS pointers. This is ignored if NotInAll is set to `true`.
repeated bytes not_in_refs = 5;
}
......@@ -116,4 +147,5 @@
repeated bytes not_in_refs = 5;
}
// GetNewLFSPointersResponse is a response for the GetNewLFSPointers RPC.
message GetNewLFSPointersResponse {
......@@ -119,4 +151,5 @@
message GetNewLFSPointersResponse {
// LfsPointers is the list of LFS pointers which were requested.
repeated LFSPointer lfs_pointers = 1;
}
......@@ -120,4 +153,5 @@
repeated LFSPointer lfs_pointers = 1;
}
// GetAllLFSPointersRequest is a request for the GetAllLFSPointers RPC.
message GetAllLFSPointersRequest {
......@@ -123,5 +157,7 @@
message GetAllLFSPointersRequest {
// Repository is the repository for which LFS pointers shoul be retrieved
// from.
Repository repository = 1[(target_repository)=true];
reserved 2;
}
......@@ -124,5 +160,6 @@
Repository repository = 1[(target_repository)=true];
reserved 2;
}
// GetAllLFSPointersResponse is a response for the GetAllLFSPointers RPC.
message GetAllLFSPointersResponse {
......@@ -128,3 +165,4 @@
message GetAllLFSPointersResponse {
// LfsPointers is the list of LFS pointers.
repeated LFSPointer lfs_pointers = 1;
}
......@@ -132,4 +132,6 @@
uint64 repl_job_id = 4;
// If the reference storage was not specified, reply with the reference used
string reference_storage = 5;
// The list of errors that appeared during the operation execution for the current repository.
repeated string errors = 6;
}
......@@ -40,11 +40,6 @@
op: ACCESSOR
};
}
rpc ListRemotes(ListRemotesRequest) returns (stream ListRemotesResponse) {
option (op_type) = {
op: ACCESSOR
};
}
}
message AddRemoteRequest {
......
......@@ -205,6 +205,9 @@
op: MUTATOR
};
}
// RemoveRepository will move the repository to `+gitaly/tmp/<relative_path>_removed` and
// eventually remove it. This ensures that even on networked filesystems the
// data is actually removed even if there's someone still handling the data.
rpc RemoveRepository(RemoveRepositoryRequest) returns (RemoveRepositoryResponse) {
option (op_type) = {
op: MUTATOR
......@@ -625,10 +628,6 @@
message Remote {
// url is the URL of the remote repository.
string url = 1;
// name is the name of the remote repository. This is mainly used to
// determine where fetched references should end up, e.g. in
// `refs/remotes/$name/`.
string name = 2;
// http_authorization_header is the HTTP header which should be added to
// the request in order to authenticate against the repository.
string http_authorization_header = 3;
......@@ -644,6 +643,15 @@
//
// If no refspecs are given, this defaults to "all_refs".
repeated string mirror_refmaps = 4;
// Previously, it was possible to specify a remote name. This was quite a
// dangerous field to set though: the name was simply used to create an ad-hoc
// remote which got deleted afterwards again. So unexpectedly, the remote
// isn't retained. And second, if the user chose the name of an existing
// remote, then it would've been deleted after the call. So in effect, the
// field was at best confusing and useless and at worst actively harmful.
reserved 2;
reserved "name";
}
message GetObjectDirectorySizeRequest {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment