Deuxfleurs/garage
48
56
Fork 73
Code Issues 99 Pull requests 13 Releases 30 Wiki Activity

Merge pull request 'Fix span name for api server requests' (#392) from fix-span-name into main
All checks were successful
continuous-integration/drone/push Build is passing
Details

Browse source 
Reviewed-on: #392
This commit is contained in:
Alex 2022-09-26 16:57:37 +02:00
commit ea7a571d88
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/api/generic_server.rs
View file

@ -174,7 +174,11 @@ impl<A: ApiHandler> ApiServer<A> {
let current_context = Context::current(); let current_context = Context::current();
let current_span = current_context.span(); let current_span = current_context.span();
current_span.update_name::<String>(format!("S3 API {}", endpoint.name())); current_span.update_name::<String>(format!(
"{} API {}",
A::API_NAME_DISPLAY,
endpoint.name()
));
current_span.set_attribute(KeyValue::new("endpoint", endpoint.name())); current_span.set_attribute(KeyValue::new("endpoint", endpoint.name()));
endpoint.add_span_attributes(current_span); endpoint.add_span_attributes(current_span);