From 2770dbbf11a4bbaef2d3b3807efbf61b64c4aec2 Mon Sep 17 00:00:00 2001 From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> Date: Fri, 14 Aug 2026 20:23:11 +0100 Subject: [PATCH] docs(telemetry): drop the legacy daemon name from the workload README The rename check rewrites a bare pre-rename binary name in any processed doc, which turned the sampler's selector description into "against xrpld or xrpld". Describe the fallback without spelling the legacy token. --- docker/telemetry/workload/README.md | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/docker/telemetry/workload/README.md b/docker/telemetry/workload/README.md index 53f4b17415..9221b47713 100644 --- a/docker/telemetry/workload/README.md +++ b/docker/telemetry/workload/README.md @@ -314,12 +314,13 @@ per leg; it is rarely run by hand. ./collect_system_metrics.sh 5020,5021,5022 300 /tmp/metrics.json ``` -Processes are selected by matching `argv[0]`'s basename against `xrpld` or -`rippled`. A wrapper that merely names the binary in its arguments, and -unrelated tools whose command line happens to contain the string, are not -sampled — including them diluted the CPU average and attributed a foreign -process's RSS to the node. `ps -C xrpld` is not usable for this: xrpld renames -itself, so its `comm` is `xrpld-main`. +Processes are selected by matching `argv[0]`'s basename against the daemon +binary name; the pre-rename spelling is accepted too, so the sampler still +works against an older deployment. A wrapper that merely names the binary in +its arguments, and unrelated tools whose command line happens to contain the +string, are not sampled — including them diluted the CPU average and +attributed a foreign process's RSS to the node. `ps -C xrpld` is not usable +for this: xrpld renames itself, so its `comm` is `xrpld-main`. Selection covers the whole host, so a second xrpld from another checkout is sampled as well. Benchmark on a machine running one cluster only.