Activity

Today
We've considered porting these builders but decided not to at this time. We have a standard Debian builder in many run-mod variations, and a Sid builder which aims to use a newer Linux kernel version…
Yesterday
Thanks. (Moving myself to CC since this is already reviewed, and I don't have comments.)
See https://ci.chromium.org/b/8715046913413654785 for example: ``` === RUN TestObjdumpRISCV64TestDecodeGNUSyntaxdata panic: test timed out after 10m0s running tests: TestObjdumpRISCV64TestDecod…
See https://ci.chromium.org/b/8715046724760256497 for example: ``` === RUN TestMarshalUnmarshal --- FAIL: TestMarshalUnmarshal (0.00s) panic: interface conversion: *sha3.asmState is not encoding.B…
Thanks.
Stopping this legacy trybot run since it matched 0 legacy builders. Note that the legacy slowbot syntax needs to be in a comment, not commit message. But that shouldn't matter now that #67307 is don…
These builders have been migrated to LUCI. For golang/go#67307.
The previous build infrastructure had a -race builder for linux/s390x, so port it over too. Notably, the previous configuration enabled the -race configuration on the main branch of the main Go repo…
It's working as expected in x/ repos too. There's one test failure in x/arch and x/crypto, but the same happens on the old infra builders, so it's a port problem and not a builder problem. It can be …
(1 comment) I agree that if it's possible to use a fixed configuration that works in both cases, it would be better. Adding a flag creates work for the users to pass in the right flag value dependin…
Thanks. This entry doesn't exist in the gophers package yet and needs to be added, right? You can check by running `go test ./devapp/owners` locally. It doesn't matter too much, but perhaps this sh…
Great, thank you. I think all that's left on our side is to adjust the builder configuration to let the builder run on applicable golang.org/x repositories (it was set to try only the main Go repo w…
The builder is running in the main Go repo. Remove the temporary skip of golang.org/x repos as the next step of its onboarding. Also move it to LOW_CAPACITY_HOSTS since by now its capacity is known …
Last Week
Glad to hear that, and congrats on getting to this point! The swarming bot generally takes care of using all of the CPU cores and machine capacity available to it, so the typical setup is to have on…
Closing because it turns out these were left overs from testing rather than real Go release versions that need to be handled. See https://go.dev/issue/68634#issuecomment-2867535846. Thanks.
I've had a chance to talk with Russ about this, and he confirmed that these "go1.8.5rc5" and "go1.9.2rc2" versions aren't really proper Go releases—they're side-effects that happened to be produced…
We've taken the alternative route of making the generated development versions be compatible with the existing https://go.dev/doc/toolchain#name format (as a "non-standard toolchain") in #73372. Is t…
Copied Votes: * Code-Review+1, Code-Review+2 (copy condition\*: "**is:ANY**") * LUCI-TryBot-Result+1 (copy condition: "**changekind:NO_CODE_CHANGE**") \* The label has `labelCopyEnforcement` or `lab…
Apply an analogous change for GenerateAutoSubmitChange to mirror what CL 670475 did in CreateAutoSubmitChange. For golang/go#68276.
Copied Votes: * Code-Review+1, Code-Review+2 (copy condition\*: "**is:ANY**") * LUCI-TryBot-Result+1 (copy condition: "**changekind:NO_CODE_CHANGE**") \* The label has `labelCopyEnforcement` or `lab…
I left some review comments from a quick look, I hope they're helpful. I haven't tried to test the changes locally so I may have missed something. In general, the test seems to be useful, but I th…
Thanks. Yes, that's right. It has some nice properties like not requiring the caller to know the modified files (if any) in advance, and automatically not mailing a CL if nothing changed. CreateAuto…
Copied Votes: * Code-Review+1 (copy condition\*: "**is:ANY**") Outdated Votes: * LUCI-TryBot-Result-1 (copy condition: "changekind:NO_CODE_CHANGE") \* The label has `labelCopyEnforcement` or `label…
Copied Votes: * Code-Review+1 (copy condition\*: "**is:ANY**") Outdated Votes: * LUCI-TryBot-Result+1 (copy condition: "changekind:NO_CODE_CHANGE") \* The label has `labelCopyEnforcement` or `label…
In the unexpected case that a cloud build script fails, it's generally useful to see the log that contains the commands and their output, and not the result GCS bucket. That bucket likely holds no re…
Using CloudBuild.RunScript to regenerate the config involves bookkeeping steps to determine whether config/config.json changed or not, to avoid creating (and then closing) a CL if it's not needed. G…
Does `go generate ./internal/configgen` work for you locally? When I initially tried the same change locally earlier, it still failed: ``` 2025/05/08 11:57:42 padding not defined for program "githu…
Note that this change appears to break the ability to regenerate x/telemetry/config: ``` $ go generate golang.org/x/telemetry/internal/configgen 2025/05/08 10:48:26 listing versions for "github.com/…
This issue has stopped matching because this builder in LUCI no longer has -bsiegert in the name (it's still there in the underlying bot name, but the pattern uses builder name), but there've been so…
This issue has stopped matching because this builder in LUCI no longer has -bsiegert in the name (it's still there in the underlying bot name, but the pattern uses builder name), but there've been so…
The netbsd-amd64-8_0 builder doesn't exist by now (LUCI has 10.0 and 10.1, old infra has 9.3), closing.
``` #!watchflakes default <- builder ~ `netbsd-arm64` && (log ~ `SIGQUIT` || log ~ `test timed out` || `failed to start: context deadline exceeded`) ``` Filing this to hopefully catch future instanc…
I believe by now we have resolved the general approach to support Plan 9 testing on LUCI. The plan9 arm and 386 builders seem to be generally working okay for a while now. The amd64 builder seems to …
I believe by now we have resolved the general approach to support Plan 9 testing on LUCI. The plan9 arm and 386 builders seem to be generally working okay for a while now. The amd64 builder seems to …
I'm not sure why this issue was opened 9 hours ago. Given #72857 is now resolved, this should happen only if the builder is dead for 24 hrs, but from looking at [its task history](https://chromium-sw…
It seems these missing builders need some time before they're back. Annotate them with tracking issues so that there's less purple on the gotip post-submit console. For golang/go#73568. For golang/g…
Noting that we ran into the same error today. (CC @cherrymui.) The app once again ended up becoming "standalone" for reasons unknown to us. We re-added it to the project.
This is an enhancement to speed up the release process and slightly reduce its resource use, by adding some complexity to its implementation that lets it avoid duplicate work. The Go build system us…
By now the bot is back and idle, having run the last 25+ builds successfully. Not sure that there's anything to do here.
Earlier
There are currently no bots running at https://ci.chromium.org/ui/p/golang/builders/ci/gotip-openbsd-riscv64. CC @4a6f656c, @golang/riscv64.
There are currently no bots running at https://ci.chromium.org/ui/p/golang/builders/ci/gotip-freebsd-riscv64. CC @mengzhuo, @golang/riscv64.
From looking at the source for those functions ([here](https://source.chromium.org/chromium/infra/infra_superproject/+/main:infra/luci/appengine/swarming/swarming_bot/api/platforms/linux.py?q=%22%5B%…
CC @golang/mips.
@xnox Including additional fields about the toolchain and its possible modifications in the build information (in addition to the go version string) like that is an interesting idea. It has some conn…
It would be good to mention this in Go 1.25 release notes (#71661), otherwise users may not know about it. See [doc/README.md](https://cs.opensource.google/go/go/+/master:doc/README.md).
These build failures are fixed via revert CLs above. Closing.
dmitshur starred github.com/apple/eidas3w
We haven't seen this happen for a while, right? I believe this was a combination of some short-lived builder VMs inadvertently staying around for much longer than intended, and some cache directorie…
It seems to be generally working well. Let's call this done; we can reopen if more is needed.
We'll need to create a backport candidate issue for the Go1.24.3 milestone; moving this to Go1.25.
@srinivas-pokala I believe the change in [crrev.com/c/6439429](https://crrev.com/c/6439429) is rolled out and you should be able to try starting up the builder again.
Fixed by [CL 664495](https://go.dev/cl/664495).
Closed by merging [CL 665856](https://go.dev/cl/665856) (commit golang/build@48b27723f5bd8d92de3f0fff371a73debc7d8feb) to `luci-config`.
The new issue #73405 (CC @prattmic) is a continuation of this, folding this into that.
The new issue #73405 (CC @prattmic) is a continuation of this, folding this into that.
We can try moving the sid builder (#61112) to cos-dev (up from cos-113-lts). If that ends up being too new and too noisy to be useful, we can move down to something less bleeding edge like cos-beta o…
Fixed by [CL 665717](https://go.dev/cl/665717).
Yes, agreed, we should apply this to cmd/dist too. Retitled to capture that too. As for why golangbuild (and coordinator, previously) generates a VERSION file, it does it for a few related reasons: …
July 4, 2024. Today I got a chance to try out the Apple Vision Pro. :)
dmitshur pushed to main in dmitri.shuralyov.com/scratch1y
5eb48b5f5d31d39777719edb35d37b724f183dc8quine: add load-bearing trailing space
dmitshur created package in dmitri.shuralyov.com/scratch/quine1y
LGTM. Thanks, and congrats on your first new Go package here! Done in PS 2. Patch set 2 seems to resolve this.
Thanks! If it's ready to be submitted from your side, I can do that now.
dmitshur pushed to main in dmitri.shuralyov.com/html/belt1y
d8009a3578d1be4dafb93e9ee4894d2bbd2b5e3badd OnClick field to CommitID
63f3644d8635396fd13789affe363ee9f693e8ecadd dark mode support in Reference
They are. Although Go can make them too easy via `go:embed`. See some initial comments. I suggest either adding a quite that can be licensed via the existing BSD-3-Clause LICENSE at the top level o…
Thanks! Congrats on your first change. :D See one inline comment that's unresolved (yellow border). You can leave a draft comment like "Done." and mark the 'Resolved' checkbox, then when you upload …
dmitshur pushed to main in dmitri.shuralyov.com/go/generated2y
e1de015411532fa144166aada4201ee443800425add test case with no final newline
6904a2a3d88d6c3ad39737a7c5bf226f5ebbd482modernize test code slightly
dmitshur pushed to main in dmitri.shuralyov.com/gpu/mtl2y
85de2813cf6bd8d6d2f0240fdebf943cbffc2b8afix "supportsFamily not found" warning on macOS 10.13 and 10.14
dmitshur pushed to main in dmitri.shuralyov.com/gpu/mtl2y
b78c9fe623cca0431d866c9e206d3f83665bcf8bdeprecate FeatureSet and Device.SupportsFeatureSet
2a3c46a829b92964146abf7f6c2d649cd66cce97add GPUFamily and Device.SupportsFamily
I agree with the points made above. This entry tried to say "use this language feature when it's appropriate; don't use it when it's not", which can be said for any language feature. I'll close this …
dmitshur pushed to master in dmitri.shuralyov.com/service/change2y
67e22b2f18a260a33a98c6a7d601726a58c53767maintner: update for GitCommit API change
Go fuzzing support is out of beta and no longer uses this build constraint. It should be replaced with `go1.18` by now. Done in PS 3. I'm following the Go style for commit messages, which chooses…
Shortened 'golang.org' to 'go.dev' here, and on line 24.
dmitshur pushed to master in dmitri.shuralyov.com/website/gido3y
f460621dc784d8a8f3a1593a068115b62baf214dadd support for multiple comma-separated import path patterns
be77a538f6a64b287e073b59d1b9f7e1314cf3a9upgrade to go-git v5
b5ce3c19e6b977800ee3c80bc722192f96ef657breplace mention of godoc.org with pkg.go.dev
dmitshur pushed to master in dmitri.shuralyov.com/gpu/mtl4y
28db891af037715d8c1deec7652485a173c60e25all: add darwin/arm64 (macOS) support
dmitshur pushed to master in dmitri.shuralyov.com/website/gido4y
d2c7b7eec450806980d4eefb626ccf6df2ef99dcupdate release branches to be for Go 1.15 and 1.14
dmitshur pushed to master in dmitri.shuralyov.com/website/gido4y
a78bd3c7abc465e1864ecadce67a826711a27caeadd support for "golang.org/x/pkgsite/..." issues
dmitshur pushed to master in dmitri.shuralyov.com/html/belt4y
Resolved a while ago in [commit `431bb22d`](https://github.com/shurcooL/issuesapp/commit/431bb22d86ad0a95db0780e27ab6221b49238464).
I've decided to publish this post on the [blog front page](https://dmitri.shuralyov.com/blog) now, so re-opening.
dmitshur pushed to master in dmitri.shuralyov.com/website/gido5y
b7141fef43ec3cefbd93cc444ab1f27bdc0a7725add support for "golang.org/dl/..." issues
1fdfe09b78364e4305cfc7fec4b2888fac391bb3update release branches to be for Go 1.14 and 1.13
dmitshur pushed to master in dmitri.shuralyov.com/go/prefixtitle5y
500817462677553a6769e268d66193c36b423cd6add support for "golang.org/dl/..." issues