Skip to content

Commit

Permalink
Emit a span for get_zarr_manifest()
Browse files Browse the repository at this point in the history
  • Loading branch information
jwodder committed Jul 26, 2024
1 parent 266e8db commit f885fc9
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 2 deletions.
2 changes: 2 additions & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ tower-http = { version = "0.5.2", features = ["set-header", "trace"] }
tracing = "0.1.40"
tracing-subscriber = { version = "0.3.18", features = ["json", "local-time", "time"] }
url = { version = "2.5.2", features = ["serde"] }
uuid = { version = "1.10.0", features = ["fast-rng", "v4"] }
xml-rs = "0.8.20"

[dev-dependencies]
Expand Down
3 changes: 1 addition & 2 deletions src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ use axum::{
};
use clap::Parser;
use std::fmt;
use std::io::stderr;
use std::net::IpAddr;
use std::sync::Arc;
use tower::service_fn;
Expand Down Expand Up @@ -78,7 +77,7 @@ fn main() -> anyhow::Result<()> {
tracing_subscriber::fmt::layer()
.json()

Check warning on line 78 in src/main.rs

View check run for this annotation

Codecov / codecov/patch

src/main.rs#L78

Added line #L78 was not covered by tests
.with_timer(timer)
.with_writer(stderr),
.with_writer(std::io::stderr),

Check warning on line 80 in src/main.rs

View check run for this annotation

Codecov / codecov/patch

src/main.rs#L80

Added line #L80 was not covered by tests
)
.with(
Targets::new()
Expand Down
1 change: 1 addition & 0 deletions src/zarrman/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,7 @@ impl ZarrManClient {

/// Retrieve the Zarr manifest at the given [`ManifestPath`] in the
/// manifest tree, either via an HTTP request or from a cache
#[tracing::instrument(skip_all, fields(id = %uuid::Uuid::new_v4(), manifest = %path))]

Check warning on line 278 in src/zarrman/mod.rs

View check run for this annotation

Codecov / codecov/patch

src/zarrman/mod.rs#L278

Added line #L278 was not covered by tests
async fn get_zarr_manifest(
&self,
path: &ManifestPath,
Expand Down

0 comments on commit f885fc9

Please sign in to comment.