Skip to content

Commit f62d320

Browse files
author
Bryan C. Mills
committed
cmd/go/internal/renameio: include ios in the darwin test-flake mitigation
Because the "ios" build constraint implies "darwin", it is already included in the general "darwin" flakiness workaround in cmd/go/internal/robustio. We just need to relax the renameio test to avoid false-positives there. I do not expect this change to drive the rate of false-positives down to zero, but it should at least reduce noise on the build dashboard. For #42066 Change-Id: Ia33dbd33295fce5b3261b4831f2807ce29b82e65 Reviewed-on: https://go-review.googlesource.com/c/go/+/263777 Trust: Bryan C. Mills <[email protected]> Reviewed-by: Cherry Zhang <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent 05f5ae7 commit f62d320

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/cmd/go/internal/renameio/renameio_test.go

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -144,10 +144,12 @@ func TestConcurrentReadsAndWrites(t *testing.T) {
144144
// As long as those are the only errors and *some* of the reads succeed, we're happy.
145145
minReadSuccesses = attempts / 4
146146

147-
case "darwin":
148-
// The filesystem on macOS 10.14 occasionally fails with "no such file or
149-
// directory" errors. See https://golang.org/issue/33041. The flake rate is
150-
// fairly low, so ensure that at least 75% of attempts succeed.
147+
case "darwin", "ios":
148+
// The filesystem on certain versions of macOS (10.14) and iOS (affected
149+
// versions TBD) occasionally fail with "no such file or directory" errors.
150+
// See https://golang.org/issue/33041 and https://golang.org/issue/42066.
151+
// The flake rate is fairly low, so ensure that at least 75% of attempts
152+
// succeed.
151153
minReadSuccesses = attempts - (attempts / 4)
152154
}
153155

0 commit comments

Comments
 (0)