x/build: builder issues bringing up new ports #61925
The openbsd/ppc64 port is being merged (per #56001) - however, CL 482115 (d0099ef) marked the port as "broken" since it is incomplete. This has a flow on effect that the builder is now effectively useless - regular build runs now immediately fail with:
And attempting to use it via
The only way to get the builder to actually run, appears to be to run it with a CL that removes
This largely defeats the point of having a builder online prior to a merge starting - while it has been normal to mark the builder has having known issues, in the past one has been able to identify what state the port is in (i.e. where it is currently failing) based on the normal https://build.golang.org dashboard. Likewise, testing an actual work-in-progress port and ensuring that a change works as expected (i.e. progresses) on the port builder has been possible in the past.
I'm not sure what is expected here - are we able to remove
The text was updated successfully, but these errors were encountered:
I marked them broken because the release process is now packaging all ports regardless of whether they pass tests, and I used broken-ness to indicate that we shouldn't try. From my perspective there's basically no difference between a port that simply fails to compile and one that fails because it's marked broken, which is why I didn't think too hard about doing it.
I think you should feel free to mark them unbroken as soon as they compile. (I assume you don't need the trybots to tell you when that is, but maybe I'm wrong?)
Understood. And I tend to agree with that.
The issue here is the lack of data from the dashboard and trybots - for example, compare:
I think the general expectation is that the broken flag is removed at the top of a CL stack to get a port working, and the builders begin to work then. If it is needed to see builder results before that point, adding the -broken flag to its configuration, along with marking it with a known issue, makes sense.
This should allow the dashboard and trybots to still provide useful results. Also add known issues for openbsd/mips64. Updates golang/go#58110 Updates golang/go#61925 Change-Id: I102f4e913cddc38dc617972d1da8cc291cda0a0b Reviewed-on: https://go-review.googlesource.com/c/build/+/518655 Run-TryBot: Joel Sing <firstname.lastname@example.org> Reviewed-by: Heschi Kreinick <email@example.com> Reviewed-by: Dmitri Shuralyov <firstname.lastname@example.org> Reviewed-by: Dmitri Shuralyov <email@example.com> TryBot-Result: Gopher Robot <firstname.lastname@example.org>