[exporter/splunkhec] enabled http2 healthcheck

.chloggen/splunkhec-exporter-http2.yaml

@@ -0,0 +1,27 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: enhancement
+
+# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
+component: splunkhecexporter
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Enable HTTP/2 health check by default
+
+# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
+issues: [29717]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:
+
+# If your change doesn't affect end users or the exported elements of any package,
+# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
+# Optional: The change log or logs in which this entry should be included.
+# e.g. '[user]' or '[user, api]'
+# Include 'user' if the change is relevant to end users.
+# Include 'api' if there is a change to a library API.
+# Default: '[user]'
+change_logs: []

exporter/splunkhecexporter/README.md

@@ -29,6 +29,10 @@ The following configuration options can also be configured:
 - `use_multi_metric_format` (default: false): Combines metrics with the same metadata to reduce ingest using the [multiple-metric JSON format](https://docs.splunk.com/Documentation/Splunk/9.0.0/Metrics/GetMetricsInOther#The_multiple-metric_JSON_format). Applicable in the `metrics` pipeline only.
 - `disable_compression` (default: false): Whether to disable gzip compression over HTTP.
 - `timeout` (default: 10s): HTTP timeout when sending data.
+- `http2_read_idle_timeout` (default = 10s): Send a ping frame for a health check if the connection has been idle for the configured value.
+  0s means http/2 health check will be disabled.
+- `http2_ping_timeout` (default = 10s): Triggered by `http2_read_idle_timeout`; When there's no response to the ping within the configured value,
+  the connection will be closed. If this value is set to 0, it will default to 15s.
 - `insecure_skip_verify` (default: false): Whether to skip checking the certificate of the HEC endpoint when sending data over HTTPS.
 - `ca_file` (no default) Path to the CA cert to verify the server being connected to.
 - `cert_file` (no default) Path to the TLS cert to use for client connections when TLS client auth is required.

exporter/splunkhecexporter/config_test.go

@@ -70,9 +70,11 @@ func TestLoadConfig(t *testing.T) {
 						},
 						InsecureSkipVerify: false,
 					},
-					MaxIdleConns:        &hundred,
-					MaxIdleConnsPerHost: &hundred,
-					IdleConnTimeout:     &idleConnTimeout,
+					MaxIdleConns:         &hundred,
+					MaxIdleConnsPerHost:  &hundred,
+					IdleConnTimeout:      &idleConnTimeout,
+					HTTP2ReadIdleTimeout: 10 * time.Second,
+					HTTP2PingTimeout:     10 * time.Second,
 				},
 				RetrySettings: exporterhelper.RetrySettings{
 					Enabled:             true,

exporter/splunkhecexporter/factory.go

@@ -20,10 +20,12 @@ import (
 )
 
 const (
-	defaultMaxIdleCons     = 100
-	defaultHTTPTimeout     = 10 * time.Second
-	defaultIdleConnTimeout = 10 * time.Second
-	defaultSplunkAppName   = "OpenTelemetry Collector Contrib"
+	defaultMaxIdleCons          = 100
+	defaultHTTPTimeout          = 10 * time.Second
+	defaultHTTP2ReadIdleTimeout = time.Second * 10
+	defaultHTTP2PingTimeout     = time.Second * 10
+	defaultIdleConnTimeout      = 10 * time.Second
+	defaultSplunkAppName        = "OpenTelemetry Collector Contrib"
 )
 
 // TODO: Find a place for this to be shared.
@@ -55,10 +57,12 @@ func createDefaultConfig() component.Config {
 		LogDataEnabled:       true,
 		ProfilingDataEnabled: true,
 		HTTPClientSettings: confighttp.HTTPClientSettings{
-			Timeout:             defaultHTTPTimeout,
-			IdleConnTimeout:     &defaultIdleConnTimeout,
-			MaxIdleConnsPerHost: &defaultMaxConns,
-			MaxIdleConns:        &defaultMaxConns,
+			Timeout:              defaultHTTPTimeout,
+			IdleConnTimeout:      &defaultIdleConnTimeout,
+			MaxIdleConnsPerHost:  &defaultMaxConns,
+			MaxIdleConns:         &defaultMaxConns,
+			HTTP2ReadIdleTimeout: defaultHTTP2ReadIdleTimeout,
+			HTTP2PingTimeout:     defaultHTTP2PingTimeout,
 		},
 		SplunkAppName:           defaultSplunkAppName,
 		RetrySettings:           exporterhelper.NewDefaultRetrySettings(),