diff --git a/tests/e2e/logging_test.go b/tests/e2e/logging_test.go index 3411983903c..d391e2f1b5d 100644 --- a/tests/e2e/logging_test.go +++ b/tests/e2e/logging_test.go @@ -37,7 +37,12 @@ func TestNoErrorLogsDuringNormalOperations(t *testing.T) { e2e.WithClusterSize(1), e2e.WithLogLevel("debug"), }, - allowedErrors: map[string]bool{"setting up serving from embedded etcd failed.": true}, + allowedErrors: map[string]bool{ + "setting up serving from embedded etcd failed.": true, + // See https://github.com/etcd-io/etcd/pull/19040#issuecomment-2539173800 + // TODO: Remove with etcd 3.7 + "cannot detect storage schema version: missing confstate information": true, + }, }, { name: "three node cluster", @@ -45,7 +50,12 @@ func TestNoErrorLogsDuringNormalOperations(t *testing.T) { e2e.WithClusterSize(3), e2e.WithLogLevel("debug"), }, - allowedErrors: map[string]bool{"setting up serving from embedded etcd failed.": true}, + allowedErrors: map[string]bool{ + "setting up serving from embedded etcd failed.": true, + // See https://github.com/etcd-io/etcd/pull/19040#issuecomment-2539173800 + // TODO: Remove with etcd 3.7 + "cannot detect storage schema version: missing confstate information": true, + }, }, { name: "three node cluster with auto tls (all)", @@ -57,7 +67,12 @@ func TestNoErrorLogsDuringNormalOperations(t *testing.T) { e2e.WithClientAutoTLS(true), e2e.WithClientConnType(e2e.ClientTLS), }, - allowedErrors: map[string]bool{"setting up serving from embedded etcd failed.": true}, + allowedErrors: map[string]bool{ + "setting up serving from embedded etcd failed.": true, + // See https://github.com/etcd-io/etcd/pull/19040#issuecomment-2539173800 + // TODO: Remove with etcd 3.7 + "cannot detect storage schema version: missing confstate information": true, + }, }, { name: "three node cluster with auto tls (peers)", @@ -67,7 +82,12 @@ func TestNoErrorLogsDuringNormalOperations(t *testing.T) { e2e.WithIsPeerTLS(true), e2e.WithIsPeerAutoTLS(true), }, - allowedErrors: map[string]bool{"setting up serving from embedded etcd failed.": true}, + allowedErrors: map[string]bool{ + "setting up serving from embedded etcd failed.": true, + // See https://github.com/etcd-io/etcd/pull/19040#issuecomment-2539173800 + // TODO: Remove with etcd 3.7 + "cannot detect storage schema version: missing confstate information": true, + }, }, { name: "three node cluster with auto tls (client)", @@ -77,7 +97,12 @@ func TestNoErrorLogsDuringNormalOperations(t *testing.T) { e2e.WithClientAutoTLS(true), e2e.WithClientConnType(e2e.ClientTLS), }, - allowedErrors: map[string]bool{"setting up serving from embedded etcd failed.": true}, + allowedErrors: map[string]bool{ + "setting up serving from embedded etcd failed.": true, + // See https://github.com/etcd-io/etcd/pull/19040#issuecomment-2539173800 + // TODO: Remove with etcd 3.7 + "cannot detect storage schema version: missing confstate information": true, + }, }, } @@ -114,7 +139,7 @@ func TestNoErrorLogsDuringNormalOperations(t *testing.T) { err := json.Unmarshal([]byte(line), &entry) require.NoErrorf(t, err, "parse log line as json, line: %s", line) - if tc.allowedErrors[entry.Message] { + if tc.allowedErrors[entry.Message] || tc.allowedErrors[entry.Error] { continue } diff --git a/tests/e2e/zap_logging_test.go b/tests/e2e/zap_logging_test.go index b77574c0a27..6752a23cc2f 100644 --- a/tests/e2e/zap_logging_test.go +++ b/tests/e2e/zap_logging_test.go @@ -71,6 +71,7 @@ type logEntry struct { Timestamp string `json:"ts"` Caller string `json:"caller"` Message string `json:"msg"` + Error string `json:"error"` } func TestConnectionRejectMessage(t *testing.T) {