Skip to content
Snippets Groups Projects
Commit fc11eb64 authored by Tigran Mkrtchyan's avatar Tigran Mkrtchyan :coffee:
Browse files

frontend-grpc: use provided archiveReportURL AS-IS

The original gRPC based frontend-grpc was assuming that the Archive request will generate the archiveId.
As a result, the archiveReportURL that sent from the storage system could not include the required the
archiveId. Thus the final archiveReportURL is built on in the frontend-grpc

Since commit 13b43466 have introduced a Create step, the storage system is able to fully build the
archiveReportURL on it's side and let frontend-grpc to use the provided report URL as-is.

Closes: #144
parent 8466f8e3
Branches
Tags
No related merge requests found
......@@ -126,7 +126,7 @@ Status CtaRpcImpl::Archive(::grpc::ServerContext* context, const ::cta::frontend
archiveRequest.requester.group = request->md().cli().user().groupname();
archiveRequest.storageClass = storageClass;
archiveRequest.srcURL = request->md().transport().dst_url();
archiveRequest.archiveReportURL = request->md().transport().report_url() + "?archiveid=" + std::to_string(archiveFileId);
archiveRequest.archiveReportURL = request->md().transport().report_url();
archiveRequest.archiveErrorReportURL = request->md().transport().error_report_url();
archiveRequest.creationLog.host = context->peer();
archiveRequest.creationLog.username = instance;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment