-
Notifications
You must be signed in to change notification settings - Fork 18k
crypto/x509: TestSystemVerify consistently failing #58791
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I think this is something to do with an expiration — all of the builders started failing at once, even on TryBots at older base CLs (https://storage.googleapis.com/go-build-log/cd3d6904/windows-386-2016_967987aa.log). |
Change https://go.dev/cl/472295 mentions this issue: |
Change https://go.dev/cl/472097 mentions this issue: |
Found new dashboard test flakes for:
2023-02-28 19:35 windows-amd64-2016 go@4a9c84d0 crypto/x509.TestSystemVerify (log)
2023-02-28 19:35 windows-arm64-11 go@4a9c84d0 crypto/x509.TestSystemVerify (log)
2023-02-28 21:59 windows-amd64-race go@ec26277a crypto/x509.TestSystemVerify (log)
2023-02-28 22:18 windows-amd64-2016 go@65fa9ce8 crypto/x509.TestSystemVerify (log)
2023-02-28 22:18 windows-amd64-race go@65fa9ce8 crypto/x509.TestSystemVerify (log)
2023-02-28 22:18 windows-arm64-11 go@65fa9ce8 crypto/x509.TestSystemVerify (log)
2023-02-28 23:42 windows-amd64-2016 go@0384235a crypto/x509.TestSystemVerify (log)
2023-02-28 23:42 windows-amd64-race go@0384235a crypto/x509.TestSystemVerify (log)
2023-02-28 23:42 windows-arm64-11 go@0384235a crypto/x509.TestSystemVerify (log)
|
@gopherbot Please backport at minimum the Windows text fix aspect of this issue. This is needed to fix failing tests on release branches. |
Backport issue(s) opened: #58810 (for 1.19), #58811 (for 1.20). Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://go.dev/wiki/MinorReleases. |
Change https://go.dev/cl/472496 mentions this issue: |
Convert TestUnknownAuthorityError to use subtests, avoiding continuing the test after an unrecoverable failure. Skip TestIssue51759 on pre-macOS 11 builders, which don't enforce the behavior we were testing for. Updates #58791 Fixes #58812 Change-Id: I4e3e5bc371aa139d38052184c8232f8cb564138f Reviewed-on: https://go-review.googlesource.com/c/go/+/472496 TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Roland Shoemaker <roland@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Change https://go.dev/cl/472615 mentions this issue: |
Change https://go.dev/cl/472616 mentions this issue: |
Change https://go.dev/cl/472618 mentions this issue: |
Change https://go.dev/cl/472617 mentions this issue: |
…ntermediate handling On Windows, replace tests which rely on a root that expired last year. On Darwin fix an test which wasn't testing the expected behavior, and fix the behavior which was broken. Updates #58791 Fixes #58811 Change-Id: I771175b9e123b8bb0e4efdf58cc2bb93aa94fbae Reviewed-on: https://go-review.googlesource.com/c/go/+/472295 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Bryan Mills <bcmills@google.com> Run-TryBot: Roland Shoemaker <roland@golang.org> (cherry picked from commit bb8f9a6) Reviewed-on: https://go-review.googlesource.com/c/go/+/472616 Auto-Submit: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Convert TestUnknownAuthorityError to use subtests, avoiding continuing the test after an unrecoverable failure. Skip TestIssue51759 on pre-macOS 11 builders, which don't enforce the behavior we were testing for. Also only enable the test on builders. Updates #58791 Updates #58812 Fixes #58811 Change-Id: I4e3e5bc371aa139d38052184c8232f8cb564138f Reviewed-on: https://go-review.googlesource.com/c/go/+/472496 TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Roland Shoemaker <roland@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> (cherry picked from commit cf3d065) Reviewed-on: https://go-review.googlesource.com/c/go/+/472618 Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
…ntermediate handling On Windows, replace tests which rely on a root that expired last year. On Darwin fix an test which wasn't testing the expected behavior, and fix the behavior which was broken. Updates #58791 Fixes #58810 Change-Id: I771175b9e123b8bb0e4efdf58cc2bb93aa94fbae Reviewed-on: https://go-review.googlesource.com/c/go/+/472295 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Bryan Mills <bcmills@google.com> Run-TryBot: Roland Shoemaker <roland@golang.org> (cherry picked from commit bb8f9a6) Reviewed-on: https://go-review.googlesource.com/c/go/+/472615 Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Convert TestUnknownAuthorityError to use subtests, avoiding continuing the test after an unrecoverable failure. Skip TestIssue51759 on pre-macOS 11 builders, which don't enforce the behavior we were testing for. Also only enable the test on builders. Updates #58791 Updates #58812 Fixes #58810 Change-Id: I4e3e5bc371aa139d38052184c8232f8cb564138f Reviewed-on: https://go-review.googlesource.com/c/go/+/472496 TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Roland Shoemaker <roland@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> (cherry picked from commit cf3d065) Reviewed-on: https://go-review.googlesource.com/c/go/+/472617 Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Found new dashboard test flakes for:
2023-03-01 18:06 windows-arm64-11 go@600d372f crypto/x509.TestSystemVerify (log)
|
Thanks watchflakes, but commit 600d37 is before the fix in commit 21e451e. |
Found new dashboard test flakes for:
2023-03-01 17:28 windows-386-2016 go@bdd86bda crypto/x509.TestSystemVerify (log)
2023-03-01 17:28 windows-amd64-2016 go@bdd86bda crypto/x509.TestSystemVerify (log)
2023-03-01 17:28 windows-arm64-11 go@bdd86bda crypto/x509.TestSystemVerify (log)
2023-03-01 18:06 windows-386-2016 go@d2d0ee20 crypto/x509.TestSystemVerify (log)
2023-03-01 18:06 windows-amd64-2016 go@d2d0ee20 crypto/x509.TestSystemVerify (log)
2023-03-01 18:06 windows-arm64-11 go@d2d0ee20 crypto/x509.TestSystemVerify (log)
|
Found new dashboard test flakes for:
2023-03-01 17:28 windows-amd64-longtest go@bdd86bda crypto/x509.TestSystemVerify (log)
2023-03-01 17:28 windows-amd64-race go@bdd86bda crypto/x509.TestSystemVerify (log)
2023-03-01 18:06 windows-amd64-longtest go@d2d0ee20 crypto/x509.TestSystemVerify (log)
2023-03-01 18:06 windows-amd64-race go@d2d0ee20 crypto/x509.TestSystemVerify (log)
|
…ntermediate handling On Windows, replace tests which rely on a root that expired last year. On Darwin fix an test which wasn't testing the expected behavior, and fix the behavior which was broken. Updates golang#58791 Fixes golang#58811 Change-Id: I771175b9e123b8bb0e4efdf58cc2bb93aa94fbae Reviewed-on: https://go-review.googlesource.com/c/go/+/472295 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Bryan Mills <bcmills@google.com> Run-TryBot: Roland Shoemaker <roland@golang.org> (cherry picked from commit bb8f9a6) Reviewed-on: https://go-review.googlesource.com/c/go/+/472616 Auto-Submit: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Convert TestUnknownAuthorityError to use subtests, avoiding continuing the test after an unrecoverable failure. Skip TestIssue51759 on pre-macOS 11 builders, which don't enforce the behavior we were testing for. Also only enable the test on builders. Updates golang#58791 Updates golang#58812 Fixes golang#58811 Change-Id: I4e3e5bc371aa139d38052184c8232f8cb564138f Reviewed-on: https://go-review.googlesource.com/c/go/+/472496 TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Roland Shoemaker <roland@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> (cherry picked from commit cf3d065) Reviewed-on: https://go-review.googlesource.com/c/go/+/472618 Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Found new dashboard test flakes for:
2023-02-28 18:46 windows-arm64-11 go@9aaaedbb crypto/x509.TestSystemVerify (log)
2023-02-28 18:50 windows-arm64-11 go@89c3f7c1 crypto/x509.TestSystemVerify (log)
2023-02-28 18:50 windows-amd64-longtest go@af758177 crypto/x509.TestSystemVerify (log)
2023-02-28 18:50 windows-arm64-11 go@454198ab crypto/x509.TestSystemVerify (log)
2023-03-01 18:00 windows-arm64-11 go@8417168d crypto/x509.TestSystemVerify (log)
2023-03-01 18:01 windows-amd64-2016 go@230765a1 crypto/x509.TestSystemVerify (log)
2023-03-01 18:01 windows-amd64-longtest go@230765a1 crypto/x509.TestSystemVerify (log)
2023-03-01 18:01 windows-amd64-race go@230765a1 crypto/x509.TestSystemVerify (log)
2023-03-01 18:01 windows-arm64-11 go@230765a1 crypto/x509.TestSystemVerify (log)
|
These all appear old. |
https://build.golang.org/log/97b5cac923dc9079392efd9637b9fffa8b9f7592:
(attn @golang/security @dmitshur)
The text was updated successfully, but these errors were encountered: