Skip to content

Commit

Permalink
fix(e2e): ignore error log about failed storage update
Browse files Browse the repository at this point in the history
  • Loading branch information
ghouscht committed Dec 13, 2024
1 parent 25dfc82 commit 14b5e84
Showing 1 changed file with 27 additions and 5 deletions.
32 changes: 27 additions & 5 deletions tests/e2e/logging_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,15 +37,22 @@ 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,
},
},
{
name: "three node cluster",
options: []e2e.EPClusterOption{
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
"failed to update storage version": true,
},
},
{
name: "three node cluster with auto tls (all)",
Expand All @@ -57,7 +64,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
"failed to update storage version": true,
},
},
{
name: "three node cluster with auto tls (peers)",
Expand All @@ -67,7 +79,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
"failed to update storage version": true,
},
},
{
name: "three node cluster with auto tls (client)",
Expand All @@ -77,7 +94,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
"failed to update storage version": true,
},
},
}

Expand Down

0 comments on commit 14b5e84

Please sign in to comment.