tree 184e48af76cd34b8c8360cb7be2f659df8c71465
parent 9f27f0c3b2d14564b787aed0eb286083a53b83ba
author David Chase <drchase@google.com> 1786987798 -0400
committer David Chase <drchase@google.com> 1786999717 -0700

internal/perf: add "|" to InfluxDB allowed-in-string chars

Some of the new benchmarks have excessively creative
names.  The characters added to the regexp were observed
in actual benchmark names and the queries flowed to Influxdb
and it did not crash (and it returned what was expected),
so apparently they are okay.

For example, the originally offending query string was

`AllShortest/DijkstraAllPaths_AllBetween_1000×16|0.1(828)-96`

and the header on the (fixed) benchmark chart for it was

`AllShortest/DijkstraAllPaths_AllBetween_1000×16|0.1(828)-96`

so nothing was lost in translation.

(Note the "×" which is NOT "x", yay Unicode)

Change-Id: I5ec54e31a07cd8172e990d3728ee685c322a2d49
Reviewed-on: https://go-review.googlesource.com/c/build/+/816580
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>