internal/upload: reject reports of the program built with unstable go

hyangah · hyangah · commit 90469f017328 · 2023-10-11T20:50:29.000Z
Change-Id: Id40d6ed197e737dc7cd6ed2d4652ae5b64038a14 Reviewed-on: https://go-review.googlesource.com/c/telemetry/+/534675 Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
diff --git a/internal/upload/reports.go b/internal/upload/reports.go
@@ -184,7 +184,7 @@ func createReport(start time.Time, expiryDate string, files []string, lastWeek s
 		// does the uploadConfig want this program?
 		// if so, copy over the Stacks and Counters
 		// that the uploadConfig mentions.
-		if !cfg.HasProgram(p.Program) || !cfg.HasVersion(p.Program, p.Version) {
+		if !cfg.HasGoVersion(p.GoVersion) || !cfg.HasProgram(p.Program) || !cfg.HasVersion(p.Program, p.Version) {
 			continue
 		}
 		x := &telemetry.ProgramReport{

Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,7 @@ func createReport(start time.Time, expiryDate string, files []string, lastWeek s`
`184`	`184`	`// does the uploadConfig want this program?`
`185`	`185`	`// if so, copy over the Stacks and Counters`
`186`	`186`	`// that the uploadConfig mentions.`
`187`		`- if !cfg.HasProgram(p.Program) \|\| !cfg.HasVersion(p.Program, p.Version) {`
	`187`	`+ if !cfg.HasGoVersion(p.GoVersion) \|\| !cfg.HasProgram(p.Program) \|\| !cfg.HasVersion(p.Program, p.Version) {`
`188`	`188`	`continue`
`189`	`189`	`}`
`190`	`190`	`x := &telemetry.ProgramReport{`