Skip to content

Commit 969df88

Browse files
committed
Add breakdown of full vs partial tiles to metrics
1 parent e709d75 commit 969df88

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

crates/ct_worker/src/ctlog.rs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -678,7 +678,10 @@ async fn sequence_entries(
678678
// If the data tile is full, stage it.
679679
if n % u64::from(TlogTile::FULL_WIDTH) == 0 {
680680
stage_data_tile(n, &mut edge_tiles, &mut tile_uploads, &data_tile);
681-
metrics.seq_data_tile_size.observe(data_tile.len().as_f64());
681+
metrics
682+
.seq_data_tile_size
683+
.with_label_values(&["full"])
684+
.observe(data_tile.len().as_f64());
682685
data_tile.clear();
683686
}
684687

@@ -692,7 +695,10 @@ async fn sequence_entries(
692695
// Stage leftover partial data tile, if any.
693696
if n != old_size && n % u64::from(TlogTile::FULL_WIDTH) != 0 {
694697
stage_data_tile(n, &mut edge_tiles, &mut tile_uploads, &data_tile);
695-
metrics.seq_data_tile_size.observe(data_tile.len().as_f64());
698+
metrics
699+
.seq_data_tile_size
700+
.with_label_values(&["partial"])
701+
.observe(data_tile.len().as_f64());
696702
}
697703

698704
// Produce and stage new tree tiles.

crates/ct_worker/src/metrics.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ pub(crate) struct Metrics {
2929
pub(crate) seq_delay: Histogram,
3030
pub(crate) seq_leaf_size: Histogram,
3131
pub(crate) seq_tiles: Counter,
32-
pub(crate) seq_data_tile_size: Histogram,
32+
pub(crate) seq_data_tile_size: HistogramVec,
3333

3434
pub(crate) tree_time: Gauge,
3535
pub(crate) tree_size: Gauge,
@@ -114,9 +114,10 @@ impl Metrics {
114114
r
115115
)
116116
.unwrap();
117-
let seq_data_tile_size = register_histogram_with_registry!(
117+
let seq_data_tile_size = register_histogram_vec_with_registry!(
118118
"sequencing_data_tiles_bytes",
119119
"Size of uploaded data tiles, including partials.",
120+
&["type"],
120121
vec![10_000.0, 100_000.0, 1_000_000.0],
121122
r
122123
)

0 commit comments

Comments
 (0)