Fixed unit test

Signed-off-by: Rohanraj123 <[email protected]>
jaegertracing · Dec 6, 2024 · e40a5f7 · e40a5f7
1 parent 51ab560
commit e40a5f7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 16 deletions.
diff --git a/cmd/jaeger/config-elasticsearch.yaml b/cmd/jaeger/config-elasticsearch.yaml
@@ -32,9 +32,6 @@ extensions:
     backends:
       some_storage:
         elasticsearch:
-          rollover:
-            enabled: true
-            frequency:  "168h"
           indices:
             index_prefix: "jaeger-main"
             spans:

diff --git a/plugin/storage/es/factory_test.go b/plugin/storage/es/factory_test.go
@@ -297,19 +297,8 @@ func TestConfigurationValidation(t *testing.T) {
 			wantErr: false,
 		},
 		{
-			name: "missing servers",
-			cfg: escfg.Configuration{
-				Rollover: escfg.Rollover{
-					Enabled: true,
-				},
-			},
-			wantErr: true,
-		},
-		{
-			name: "missing rollover config",
-			cfg: escfg.Configuration{
-				Servers: []string{"http://localhost:9200"},
-			},
+			name:    "missing servers",
+			cfg:     escfg.Configuration{},
 			wantErr: true,
 		},
 	}