Ori.livneh has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/390645 )

Change subject: xenon: pass --mindwidth to flamegraph.pl
......................................................................


xenon: pass --mindwidth to flamegraph.pl

The flame graphs are currently slow to download and slow to render
because the weight threshold for plotting a function is very low.
Increasing the threshold (via --mindwidth=2) causes flamegraph.pl to
omit leaf functions that have a tiny footprint. The resultant flame
graph still contains anything anyone would reasonably care about, but is
more than ten times smaller (600k -> 50k) and renders instantly.

Change-Id: I0a01c694374799718662e16a974decfaf0589ca1
---
M modules/xenon/files/xenon-generate-svgs
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  jenkins-bot: Verified
  Gilles: Looks good to me, but someone else must approve
  Ori.livneh: Looks good to me, approved



diff --git a/modules/xenon/files/xenon-generate-svgs 
b/modules/xenon/files/xenon-generate-svgs
index f2ad376..8b40f07 100755
--- a/modules/xenon/files/xenon-generate-svgs
+++ b/modules/xenon/files/xenon-generate-svgs
@@ -11,11 +11,11 @@
     mkdir -m0755 -p "$(dirname $svg)"
     [ ! -f "$svg" -o "$svg" -ot "$log" ] && {
         echo "Generating ${svg}..."
-        nice /usr/local/bin/flamegraph.pl --title="$title" "$log" | gzip -9 > 
"$svg"
+        nice /usr/local/bin/flamegraph.pl --minwidth=2 --title="$title" "$log" 
| gzip -9 > "$svg"
     }
     [ ! -f "$rsvg" -o "$rsvg" -ot "$log" ] && {
         echo "Generating ${rsvg}..."
-        nice /usr/local/bin/flamegraph.pl --reverse --colors=blue 
--title="$title - reversed" "$log" | gzip -9 > "$rsvg"
+        nice /usr/local/bin/flamegraph.pl --minwidth=2 --reverse --colors=blue 
--title="$title - reversed" "$log" | gzip -9 > "$rsvg"
     }
 done
 

-- 
To view, visit https://gerrit.wikimedia.org/r/390645
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0a01c694374799718662e16a974decfaf0589ca1
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: Gilles <gdu...@wikimedia.org>
Gerrit-Reviewer: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to