commit 1d025eeba04e0072648dfef750da25de5364ef16
author Michael Pratt <mpratt@google.com> Thu Dec 22 12:11:58 2022 -0500
committer Gopher Robot <gobot@golang.org> Thu Dec 22 17:50:15 2022 +0000
tree c898330ef03d0d64a4113e272b793354184fd001
parent cabcec969be27a4e3d3cf7b69d85ffa74e7fbf97

all: upgrade to golang.org/x/perf@3fd27c2392831a268043c8e09d4599e7f1c441d3

This fixes AllComparisonSeries to return errors rather than panicking.

For golang/go#53538.

Change-Id: I872a1a77c8aedc3827dec650c21afa7c68b540af
Reviewed-on: https://go-review.googlesource.com/c/build/+/459195
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: David Chase <drchase@google.com>
Auto-Submit: Michael Pratt <mpratt@google.com>
Run-TryBot: Michael Pratt <mpratt@google.com>

go.mod

diff --git a/go.mod b/go.mod
index 98eda5b..06137ec 100644
--- a/go.mod
+++ b/go.mod
@@ -52,7 +52,7 @@
 	golang.org/x/mod v0.6.0
 	golang.org/x/net v0.1.0
 	golang.org/x/oauth2 v0.0.0-20211104180415-d3ed0bb246c8
-	golang.org/x/perf v0.0.0-20220913151710-7c6e287988f3
+	golang.org/x/perf v0.0.0-20221222170352-3fd27c239283
 	golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4
 	golang.org/x/sys v0.1.0
 	golang.org/x/term v0.1.0

go.sum

diff --git a/go.sum b/go.sum
index 50356ee..dd02309 100644
--- a/go.sum
+++ b/go.sum
@@ -1030,6 +1030,8 @@
 golang.org/x/oauth2 v0.0.0-20211104180415-d3ed0bb246c8/go.mod h1:KelEdhl1UZF7XfJ4dDtk6s++YSgaE7mD/BuKKDLBl4A=
 golang.org/x/perf v0.0.0-20220913151710-7c6e287988f3 h1:ChUtQhjSq/uYjerTlcpLrOkD2bGMcrRuDutZVPHo3WE=
 golang.org/x/perf v0.0.0-20220913151710-7c6e287988f3/go.mod h1:UBKtEnL8aqnd+0JHqZ+2qoMDwtuy6cYhhKNoHLBiTQc=
+golang.org/x/perf v0.0.0-20221222170352-3fd27c239283 h1:MTXrvYGLsaCdRXRtex0LY6RotT5TXmQ3Ru+5d1CnPIM=
+golang.org/x/perf v0.0.0-20221222170352-3fd27c239283/go.mod h1:UBKtEnL8aqnd+0JHqZ+2qoMDwtuy6cYhhKNoHLBiTQc=
 golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sync v0.0.0-20181108010431-42b317875d0f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sync v0.0.0-20181221193216-37e7f081c4d4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=

perf/app/influx.go

diff --git a/perf/app/influx.go b/perf/app/influx.go
index ae7219c..2d470d7 100644
--- a/perf/app/influx.go
+++ b/perf/app/influx.go
@@ -268,7 +268,10 @@
 
 	// Run the comparison. We don't have any existing results so our
 	// duplicate policy doesn't matter here. Just pick replacement.
-	comparisons := builder.AllComparisonSeries(nil, benchseries.DUPE_REPLACE)
+	comparisons, err := builder.AllComparisonSeries(nil, benchseries.DUPE_REPLACE)
+	if err != nil {
+		return fmt.Errorf("failed to creation comparison series: %w", err)
+	}
 
 	const (
 		confidence = 0.95