apply @jpkrohling feedback

open-telemetry · Sep 26, 2023 · bce885d · bce885d
1 parent ce3776a
commit bce885d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/otelcol/unmarshaler.go b/otelcol/unmarshaler.go
@@ -49,7 +49,7 @@ func unmarshal(v *confmap.Conf, factories Factories) (*configSettings, error) {
 					Sampling: &telemetry.LogsSamplingConfig{
 						Enabled:    true,
 						Tick:       10 * time.Second,
-						Initial:    10,
+						Initial:    100,
 						Thereafter: 100,
 					},
 					OutputPaths:       []string{"stderr"},

diff --git a/otelcol/unmarshaler_test.go b/otelcol/unmarshaler_test.go
@@ -48,7 +48,7 @@ func TestUnmarshalEmptyAllSections(t *testing.T) {
 		Sampling: &telemetry.LogsSamplingConfig{
 			Enabled:    true,
 			Tick:       10 * time.Second,
-			Initial:    10,
+			Initial:    100,
 			Thereafter: 100,
 		},
 		DisableCaller:     zapProdCfg.DisableCaller,

diff --git a/service/service_test.go b/service/service_test.go
@@ -544,6 +544,8 @@ func newNopConfigPipelineConfigs(pipelineCfgs pipelines.Config) Config {
 				Development: false,
 				Encoding:    "console",
 				Sampling: &telemetry.LogsSamplingConfig{
+					Enabled:    true,
+					Tick:       10 * time.Second,
 					Initial:    100,
 					Thereafter: 100,
 				},