x/telemetry: `gotelemetry view` graphs have data from the future about go HOT 3 CLOSED

gotelemetry view renders x/telemetry/internal/telemetry.Report data.
According to the definition of x/telemetry/internal/telemetry.Report, the Week is the first day the report covers.

type Report struct {
	Week     string  // first day this report covers (YYYY-MM-DD)
	LastWeek string  // Week field from latest previous report uploaded
	...
}

But, it looks to me that the Uploader sets the TimeEnd (expiryDate) as the Report.Week value. (internal/upload/reports.go)

gotelemetry view's pending telemetry.Report creation also uses TimeEnd to match the uploader's logic.

I am afraid it's too late to fix the Uploader's report generation so it sets Report.Weekwith the first day of the week span.
Can we change the definition of the Report.Week to match the current implementation?

from go.

Thanks for investigating. Yes, let's update the definition of Week to match the implementation.

from go.

Change https://go.dev/cl/595962 mentions this issue: cmd/gotelemetry: fix chart x-axis time interval display

from go.