Skip to content

Commit 45dbf63

Browse files
committed
Make all heap profile logs trace (#4845)
## Motivation In a previous change, we lowered the log level for heap profile operations from INFO to TRACE to reduce log noise from frequent monitoring polls. However, we missed one log statement in the heap profile HTTP endpoint that still uses INFO level. This inconsistency means that while most heap profile operations log at TRACE level, the HTTP endpoint still creates excessive log noise with `"Serving heap profile via /debug/pprof"` messages appearing in default logs every time the endpoint is polled. ## Proposal Change the remaining log statement in `linera-metrics/src/memory_profiler.rs` from `info!` to `trace!` for the heap profile endpoint message, completing the consistency improvement started in the earlier change. ## Test Plan 1. Deploy network with memory profiling enabled 2. Verify heap profile endpoint still works (returns pprof data) 3. Observe that default logs no longer contain repetitive "Serving heap profile" messages 4. Enable TRACE logging and verify messages appear as expected 5. Confirm consistency with other heap profile logging ## Release Plan - These changes should be backported to the latest `testnet` branch, then - be released in a validator hotfix.
1 parent 78bc4cb commit 45dbf63

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

linera-metrics/src/memory_profiler.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ impl MemoryProfiler {
5151

5252
/// HTTP endpoint for heap profile - returns fresh pprof data
5353
pub async fn heap_profile() -> Result<impl IntoResponse, StatusCode> {
54-
info!("Serving heap profile via /debug/pprof");
54+
trace!("Serving heap profile via /debug/pprof");
5555

5656
match Self::collect_heap_profile().await {
5757
Ok(profile_data) => {

0 commit comments

Comments
 (0)