From a3a31ec8ed4aff33d9f5ef8358f9f74efb249f74 Mon Sep 17 00:00:00 2001 From: Daniel McCarney Date: Thu, 21 Nov 2024 13:11:07 -0500 Subject: [PATCH] cmd/dist: skip FIPS 140-3 testing This will be re-enabled by an in-progress CR. For now, ignore this test process to prevent build breakage. Change-Id: Ica1cb1c4f9844164ab32767ce2a499021d40b997 Reviewed-on: https://go-review.googlesource.com/c/go/+/630237 Reviewed-by: Dmitri Shuralyov Reviewed-by: Cherry Mui LUCI-TryBot-Result: Go LUCI Reviewed-by: Filippo Valsorda Auto-Submit: Filippo Valsorda --- src/cmd/dist/test.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go index 59a5e42b5b0..cd3c30da0b6 100644 --- a/src/cmd/dist/test.go +++ b/src/cmd/dist/test.go @@ -715,13 +715,14 @@ func (t *tester) registerTests() { // Check that all crypto packages compile (and test correctly, in longmode) with fips. if fipsSupported() { + /* TODO(cpu): Restore in #629736 // Test standard crypto packages with fips140=on. t.registerTest("GODEBUG=fips140=on go test crypto/...", &goTest{ variant: "gofips140", env: []string{"GODEBUG=fips140=on"}, - skip: "TestHandshake|TestServerResumption|TestClientAuth|TestRenegotiate", // TODO(valsorda): remove once crypto/tls passes + skip: "TestHandshake|TestServerResumption|TestClientAuth|TestRenegotiate", // TODO(cpu): remove in #629736 pkg: "crypto/...", - }) + })*/ // Test that earlier FIPS snapshots build. // In long mode, test that they work too.