diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2021-10-18 09:18:57 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2021-11-03 09:34:38 +0300 |
commit | 2f2a001d6d224209ea0952de50d420ff5369884c (patch) | |
tree | 0de8d6b40a4efd9ba48ec9dc76c6b881aa731240 /proto/objectpool.proto | |
parent | 3b4c8b2211b024b86e3de99f38350ffd1f907417 (diff) |
objectpool: Drop UnlinkRepostioryFromObjectPool RPC
The UnlikRepostioryFromObjectPool RPC is not used by anything anymore,
and the implementation from it is dangerous given that it doesn't
actually unlink a repository from its object pool: it only tries to
remove a remote named after the pool's project path, which we wouldn't
ever create in the first place. The RPC has thus been deprecated in
release v14.3.
Remove the RPC and its backing code.
Changelog: removed
Diffstat (limited to 'proto/objectpool.proto')
-rw-r--r-- | proto/objectpool.proto | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/proto/objectpool.proto b/proto/objectpool.proto index dcbfb9a9d..e6be24c39 100644 --- a/proto/objectpool.proto +++ b/proto/objectpool.proto @@ -26,23 +26,6 @@ service ObjectPoolService { }; } - // UnlinkRepositoryFromObjectPool does not unlink the repository from the - // object pool as you'd think, but all it really does is to remove the object - // pool's remote pointing to the repository. And even this is a no-op given - // that we'd try to remove the remote by the repository's `GlRepository()` - // name, which we never create in the first place. To unlink repositories - // from an object pool, you'd really want to execute DisconnectGitAlternates - // to remove the repository's link to the pool's object database. - // - // This function is never called by anyone and highly misleading. It's thus - // deprecated and will be removed in v14.4. - rpc UnlinkRepositoryFromObjectPool(UnlinkRepositoryFromObjectPoolRequest) returns (UnlinkRepositoryFromObjectPoolResponse) { - option deprecated = true; - option (op_type) = { - op: MUTATOR - }; - } - rpc ReduplicateRepository(ReduplicateRepositoryRequest) returns (ReduplicateRepositoryResponse) { option (op_type) = { op: MUTATOR @@ -86,14 +69,6 @@ message LinkRepositoryToObjectPoolRequest { } message LinkRepositoryToObjectPoolResponse {} -// This RPC doesn't require the ObjectPool as it will remove the alternates file -// from the pool participant. The caller is responsible no data loss occurs. -message UnlinkRepositoryFromObjectPoolRequest { - Repository repository = 1 [(target_repository)=true]; // already specified as the target repo field - ObjectPool object_pool = 2 [(additional_repository)=true]; -} -message UnlinkRepositoryFromObjectPoolResponse {} - message ReduplicateRepositoryRequest { Repository repository = 1 [(target_repository)=true]; } |