Since Jan, my telemetry uploads were all rejected with 400 (Bad Request) Looking at the server, I noticed many upload reports are indeed rejected with errors like
unknown program build golang.org/x/tools/gopls@devel go1.22rc1 darwin/amd64
unknown program build golang.org/x/tools/gopls@v0.14.2 go1.21.4 darwin/arm64
Looking into the codebase further,
the server program reads configuration from a file (./config/config.json)
that is copied when the container was built https://github.com/golang/telemetry/blob/88e2d1c7f9a20e3476dd2cca5112e374c8911eed/godev/cmd/telemetrygodev/Dockerfile#L55
And, this configuration is being used until the next production release
https://github.com/golang/telemetry/blob/88e2d1c7f9a20e3476dd2cca5112e374c8911eed/godev/cmd/telemetrygodev/main.go#L40 https://github.com/golang/telemetry/blob/88e2d1c7f9a20e3476dd2cca5112e374c8911eed/godev/cmd/telemetrygodev/main.go#L51
I think the workers also share this setup.
It looks like the production service wasn't rebuilt/restarted for a while.
cc @findleyr @pjweinb
Comment From: findleyr
I think we had discussed a deployment trigger. Let's set that up, along with a push test.
Comment From: gopherbot
Change https://go.dev/cl/556860 mentions this issue: godev/cmd/telemetrygodev: add a basic URL test
Comment From: gopherbot
Change https://go.dev/cl/557855 mentions this issue: godev/cmd/telemetrygodev: add a prod deployment script
Comment From: gopherbot
Change https://go.dev/cl/558438 mentions this issue: godev/cmd/telemetrygodev: fix push test argument quoting
Comment From: gopherbot
Change https://go.dev/cl/558439 mentions this issue: godev/cmd/telemetrygodev: run push test from submodule
Comment From: findleyr
This is now done: the telemetry frontend is auto-deployed to production, following a push test, on any merge to master.