This CL isn't directly causing the failures. It's happening in builders that are new and weren't meant to run in this repo. That was fixed in CL 571737, so restarting.
It's common for Go tests that require certain capability or dependency to skip themselves if the environment doesn't meet that requirement. For example, see the various `MustHave*` helpers in [`inter…
(5 comments)
Agree it's fine for those familiar with it to just run ./main.star, but I didn't think it'd be useful to mention it. You bring up a good point that we might get contributions from where…
The enabled function encodes a policy of which builders are supported.
For example, x/mobile is classified as PT.SPECIAL and has very few
builders enabled. x/build is classified as PT.TOOL and opts o…
Thanks for adding this.
Probably fine indeed, since x/build is not very high volume. If we run into a reason to back that out, maybe a way to do that is by using a "main.star" location filter.
Let's consider starting to do that in the new-to-be v3.4/glfw module (#393). What would some of the finer details look like? I wrote the following to try to answer that and explore options.
Its fi…
v3.3/glfw is its own module, separate from the top-level module that has the older versions v3.{0,1,2}/glfw. That seems to be working well, so I expect we can repeat that pattern and also have v3.4/g…
I can confirm the same program no longer panics now that GLFW 3.3.10 is pulled in. `glfw.GetKeyName(glfw.KeyUnknown, 0x003f)` returns an empty string. I think we can close this unless there's somethi…
(2 comments)
(nit) There's usually a single space between the `]` and the original CL subject.
As documented at https://go.dev/wiki/MinorReleases#making-cherry-pick-cls, please also update the orig…
Thanks.
As a side note, I'm starting to wonder if we should instead (or in addition) link to the upstream documentation on this at https://www.glfw.org/docs/latest/compile_guide.html#compile_deps_…
The comment of the DialConfig function was dropped during CL 463097.
There doesn't seem to be a good reason to do that, so bring it back.
For golang/go#57953.
This CL targets release-branch.go1.22, is that intentional?
If so, please see https://go.dev/wiki/MinorReleases for the process of backporting, which begins with creating backport issues.
This manual parameter might become obsolete in the future, in favor of
being pulled from another source automatically. Since that hasn't been
implemented yet, add a simple automated check to it in th…
This is done, and announced at https://groups.google.com/g/golang-dev/c/CqVVuxngzDU/m/MD4qrteuAQAJ.
There may be more to do in this general, but bulk of the work is done, so I'll close this. We ca…
(2 comments)
Go 1.22 and 1.21 by now, right? (gotip will be Go 1.23.)
These are the current darwin/amd64 builder types. They all tend to specify the macOS version, so "darwin-amd64" doesn't match a…
We're aware of some issues on the LUCI post-submit dashboard (https://ci.chromium.org/p/golang/g/go-gotip/console). This is an umbrella tracking issue to collect most critical ones that affect the Go…
(1 comment)
`goDebug` is added near `goExperiment` under a comment that says “Package-scoped variables that are initialized at the start of Test.”
However it’s not initialized to any value at…
Thanks for the fix.
I think a good approach is to use the -dry-run flag and run it locally.
For example, I ran it with `-dry-run -single-pr=golang/go#65218` flags as is, and it printed:
```
Proces…
Ok, that involves more steps, please see https://go.dev/wiki/MinorReleases.
Since this is the main tracking issue, let's move it to the Go1.23 milestone.
Thanks. Adding Cherry who reviewed the new packages and is in the proposed primary owner group (runtimeTeam) to review.
Is this the right import path? The packages added in CL 548695 and CL 548676 d…
Thanks for checking in.
I've looked into this and as things stand now, there is expected additional time after the public Go 1.22.0 release, before it's available for the CIPD package building pip…
CL 567495 removed the special case that CL 528155 added temporarily.
This CL removes the updated test cases that CL 528540 needed to add
back then. They're no longer passing and that is working as in…
Post-submit builders for x/playground have been added in LUCI, for example:
https://ci.chromium.org/p/golang/g/x-playground-gotip/console
(There are more dashboards for release branches and wit…
Right now the x/playground repo is tested via the usual `go test ./...` as other x/ repos.
As described in #24823, there are tests that need additional dependencies to run (Docker).
This is the…
Thanks for reporting.
Fixing this will involve navigating the constraint that the current message is used for de-duplication, so changes to its exact text without an additional mechanism to detect…
For posterity (I didn't hit send earlier): this was deployed and confirmed to work in https://go-review.googlesource.com/c/tools/+/527455/comments/ce012492_44228720.
Thanks.
Thanks for the CL, and congrats on 1.18 dropping off!
This is generally right, but will have effect in the old infrastructure.
For LUCI trybots the change will be in `main.star` on the `luc…
The builder has been running some time and producing good signal so far.
Remove its known issue as its next step. The issue tracks the remaining
work to document its setup, and improve life-cycle edg…
(1 comment)
The replacement for this was the 10px padding of div.page being added where needed to compensate.
I'll check with another browser too to be safe.
Restore the local "dev" mode after the refactor in CL 422956.
The development environment doesn't have KubeServices set to
anything, so KubeServices.Location() was otherwise panicking.
While here, a…
(1 comment)
Pointing out that when mentioning this upcoming feature in https://go-review.googlesource.com/c/go/+/567296/comment/a8a880b9_da27e615/, I noticed that the previous approach to linking to…
(1 comment)
This URL isn't quite right. It will link to https://go.dev/reflect.Value and be 404. See https://cs.opensource.google/go/go/+/master:doc/README.md;l=30.
(Note that in a near future ther…
Thanks.
(nit) Duplicate '/'.
Some of the comments around this test talk about vendoring in the go repo generally. Perhaps it's slightly better to generalize this slightly so vendored packages have …
Thanks.
Is it worth also including `stdout.String()` here, since it seems to be unused otherwise (but may be interested to see what's there, even if nothing)?
No, purple failure and "resource exhaustion" means those builders haven't been fully added, see https://go.dev/wiki/LUCI#infra-failed--purple-failure and #60440.
This sounds interesting, can you say more about how it would work? Would it only handle one level of pagination or are you thinking about multiple levels of pagination too?
I don't know when I'll …
It sounds like this is okay to leave as is then. If in the future this change (possibly bundled with others like it) gets a higher benefit-to-cost ratio, a new issue can be filed (referencing this on…
We're getting close to the post-submit phase of the migration to LUCI, with many LUCI builders ported to https://ci.chromium.org/p/golang and ready to replace the previous coordinator-powered counter…
### Go version
go version go1.22.0 darwin/arm64
### Output of `go env` in your module/workspace:
```shell
N/A
```
### What did you do?
I used fmt's `%#v` to print a value whose typ…
Thanks.
This is covered in the mentioned issue:
> (Most x/ repos get automatic monthly updates of x/ dependencies, but x/playground is currently skipped because of #24823. So fixing that might also…
Thanks.
(nit) https://go.dev/wiki/CommitMessage suggests the first word after colon should be a verb, and a package before the colon. Probably just "cmd" is fine (like in CL 559795).
For #64169.
The previous infrastructure did not display x/playground on the
post-submit dashboard, but they did offer trybots. LUCI trybots
are required now, so it's a good time to add basic coverage for
x/playg…
Thanks.
The linux/loong64 builder [isn't yet running](https://ci.chromium.org/ui/p/golang/g/port-linux-loong64/builders) on LUCI, so let's use the previous builders.
TRY=linux-amd64-wsl, linux-loon…
> it needs some new directories and files inside the tree for its purposes
Why can’t equivalent files inside a temp directory work?
> maybe I should create them beforehand instead of doing it dur…
Tests in the standard Go library should not try to write within GOROOT itself. If writing is needed, the test should create a temp directory (e.g., in `t.TempDir()`), possibly copy files it needs the…
Consider a playground snippet that includes a go.mod file as if generated by `go mod init play.ground` using the latest stable version of Go: https://go.dev/play/p/ZtkN19wIOev.
Pressing the "Forma…