Skip to content
Merged
Show file tree
Hide file tree
Changes from 24 commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
ead2c04
switch from sapm to otlphttp as default
asreehari-splunk Nov 7, 2024
70a453a
missing update
asreehari-splunk Nov 18, 2024
1df9f57
update trace url for chef tests
asreehari-splunk Nov 18, 2024
22d4438
update trace endpoint for sapm
asreehari-splunk Nov 18, 2024
be25146
Updating zeroconfig\windows tests
pjanotti Nov 20, 2024
16eba48
zeroconfig/windows: network needs to be set as "external"
pjanotti Nov 20, 2024
a5989e0
switch from sapm to otlphttp as default
asreehari-splunk Nov 7, 2024
0367e2a
missing update
asreehari-splunk Nov 18, 2024
6205f06
update trace url for chef tests
asreehari-splunk Nov 18, 2024
0363330
update trace endpoint for sapm
asreehari-splunk Nov 18, 2024
26fc39f
Updating zeroconfig\windows tests
pjanotti Nov 20, 2024
e109170
zeroconfig/windows: network needs to be set as "external"
pjanotti Nov 20, 2024
a3fe7c1
comment out provo mirror
asreehari-splunk Nov 20, 2024
d05fd07
Merge branch 'sapm_to_otlphttp' of github.com:asreehari-splunk/splunk…
pjanotti Nov 22, 2024
f5dd888
get info about docker compose error
pjanotti Nov 22, 2024
db06c59
Add files missing to docker compose build
pjanotti Nov 22, 2024
61c1041
Build the ASP.NET app
pjanotti Nov 23, 2024
0871382
Revert commits to investigate docker compose error
pjanotti Nov 23, 2024
42b6377
Restore parts of docker-compose.yaml
pjanotti Nov 23, 2024
e4286c3
use trace_url for traces endpoint
asreehari-splunk Nov 25, 2024
798407a
Merge branch 'main' into sapm_to_otlphttp
asreehari-splunk Dec 4, 2024
f5fcd0f
Merge branch 'main' into sapm_to_otlphttp
asreehari-splunk Dec 10, 2024
d4aae4d
update test to match endpoint
asreehari-splunk Dec 11, 2024
ec80ea2
metrics endpoint is not required
asreehari-splunk Dec 12, 2024
3cbc174
Update cmd/otelcol/config/collector/ecs_ec2_config.yaml
dmitryax Dec 12, 2024
6e54e6c
update other fips configs
asreehari-splunk Dec 13, 2024
f0f3e3b
Merge branch 'main' into sapm_to_otlphttp
asreehari-splunk Dec 13, 2024
017caa3
update TA configs with otlphttp exporter
asreehari-splunk Dec 13, 2024
cb671b4
Merge branch 'main' into sapm_to_otlphttp
dmitryax Dec 16, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### 🛑 Breaking changes 🛑

- (Splunk) Change default traces exporter to otlphttp exporter. ([#5588](https://github.com/signalfx/splunk-otel-collector/pull/5588)).

## v0.114.0

### 💡 Enhancements 💡
Expand Down
20 changes: 20 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,26 @@ manually before the backward compatibility is dropped. For every configuration u
[the default agent config](https://github.com/signalfx/splunk-otel-collector/blob/main/cmd/otelcol/config/collector/agent_config.yaml)
as a reference.

### From 0.114.0 to 0.115.0

- The sapm exporter still works as before but has been deprecated. Use the otlphttp exporter instead

1. Replace the `sapm` exporter with `otlphttp` exporter using the following configuration
```yaml
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
```
2. Update traces pipeline to use otlphttp exporter instead of sapm:
```
service:
pipelines:
traces:
exporters: [otlphttp, signalfx]
```


### From 0.96.1 to 0.97.0

- `memory_ballast` is no longer effective. The garbage collection is now controlled by the soft memory limit set to 90%
Expand Down
17 changes: 9 additions & 8 deletions cmd/otelcol/config/collector/agent_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -126,9 +126,10 @@ processors:

exporters:
# Traces
sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "${SPLUNK_INGEST_URL}/v2/trace"
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
# Metrics + Events
signalfx:
access_token: "${SPLUNK_ACCESS_TOKEN}"
Expand Down Expand Up @@ -157,7 +158,7 @@ exporters:
endpoint: "${SPLUNK_INGEST_URL}/v1/log"
log_data_enabled: false
# Send to gateway
otlp:
otlp/gateway:
endpoint: "${SPLUNK_GATEWAY_URL}:4317"
tls:
insecure: true
Expand All @@ -178,15 +179,15 @@ service:
- batch
- resourcedetection
#- resource/add_environment
exporters: [sapm, signalfx]
exporters: [otlphttp, signalfx]
# Use instead when sending to gateway
#exporters: [otlp, signalfx]
#exporters: [otlp/gateway, signalfx]
metrics:
receivers: [hostmetrics, otlp, signalfx]
processors: [memory_limiter, batch, resourcedetection]
exporters: [signalfx]
# Use instead when sending to gateway
#exporters: [otlp]
#exporters: [otlp/gateway]
metrics/internal:
receivers: [prometheus/internal]
processors: [memory_limiter, batch, resourcedetection]
Expand All @@ -211,4 +212,4 @@ service:
#- resource/add_environment
exporters: [splunk_hec, splunk_hec/profiling]
# Use instead when sending to gateway
#exporters: [otlp]
#exporters: [otlp/gateway]
11 changes: 6 additions & 5 deletions cmd/otelcol/config/collector/ecs_ec2_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,10 @@ processors:

exporters:
# Traces
sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace"
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
# Metrics + Events
signalfx:
access_token: "${SPLUNK_ACCESS_TOKEN}"
Expand All @@ -129,7 +130,7 @@ exporters:
token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "${SPLUNK_INGEST_URL}/v1/log"
log_data_enabled: false

service:
extensions: [health_check, http_forwarder, zpages]
pipelines:
Expand All @@ -140,7 +141,7 @@ service:
- batch
- resourcedetection
#- resource/add_environment
exporters: [sapm, signalfx]
exporters: [otlphttp, signalfx]
metrics:
receivers: [hostmetrics, otlp, signalfx, awsecscontainermetrics]
processors: [memory_limiter, batch, filter, resourcedetection]
Expand Down
9 changes: 5 additions & 4 deletions cmd/otelcol/config/collector/fargate_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,10 @@ processors:

exporters:
# Traces
sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace"
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
# Metrics + Events
signalfx:
access_token: "${SPLUNK_ACCESS_TOKEN}"
Expand Down Expand Up @@ -123,7 +124,7 @@ service:
- batch
- resourcedetection
#- resource/add_environment
exporters: [sapm, signalfx]
exporters: [otlphttp, signalfx]
metrics:
receivers: [otlp, signalfx, awsecscontainermetrics, prometheus/internal]
processors: [memory_limiter, batch, resourcedetection]
Expand Down
6 changes: 5 additions & 1 deletion cmd/otelcol/config/collector/full_config_linux.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -637,7 +637,11 @@ exporters:
# WARN: https://ingest.${SPLUNK_REALM}.signalfx.com does NOT support gRPC at this time; use otlphttp exporter instead
otlp:
endpoint: otelcol2:4317

sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace"
sending_queue:
num_consumers: 32
# Enables the otlp http exporter
# Full configuration here: https://github.com/open-telemetry/opentelemetry-collector/tree/main/exporter/otlphttpexporter
# NOTE: These settings should be updated to the proper destination
Expand Down
13 changes: 6 additions & 7 deletions cmd/otelcol/config/collector/gateway_config.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Configuration file that uses the Splunk exporters (SAPM, SignalFx) to push
# Configuration file that uses the Splunk exporters (OTLP HTTP, SignalFx) to push
# data to Splunk products.

extensions:
Expand Down Expand Up @@ -91,11 +91,10 @@ processors:

exporters:
# Traces
sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace"
sending_queue:
num_consumers: 32
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
# Metrics + Events
signalfx:
access_token: "${SPLUNK_ACCESS_TOKEN}"
Expand Down Expand Up @@ -138,7 +137,7 @@ service:
- memory_limiter
- batch
#- resource/add_environment
exporters: [sapm]
exporters: [otlphttp]
metrics:
receivers: [otlp, signalfx]
processors: [memory_limiter, batch]
Expand Down
21 changes: 11 additions & 10 deletions cmd/otelcol/config/collector/upstream_agent_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -120,9 +120,10 @@ processors:

exporters:
# Traces
sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "${SPLUNK_INGEST_URL}/v2/trace"
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
# Metrics + Events
signalfx:
access_token: "${SPLUNK_ACCESS_TOKEN}"
Expand All @@ -143,7 +144,7 @@ exporters:
endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v1/log"
log_data_enabled: false
# Send to gateway
otlp:
otlp/gateway:
endpoint: "${SPLUNK_GATEWAY_URL}:4317"
tls:
insecure: true
Expand All @@ -162,30 +163,30 @@ service:
- batch
- resourcedetection
#- resource/add_environment
exporters: [sapm, signalfx]
exporters: [otlphttp, signalfx]
# Use instead when sending to gateway
#exporters: [otlp, signalfx]
#exporters: [otlp/gateway, signalfx]
# Required for Splunk Infrastructure Monitoring
metrics:
receivers: [hostmetrics, otlp, signalfx]
processors: [memory_limiter, batch, resourcedetection]
exporters: [signalfx]
# Use instead when sending to gateway
#exporters: [otlp]
#exporters: [otlp/gateway]
# Required for Splunk APM and/or Infrastructure Monitoring
metrics/internal:
receivers: [prometheus/internal]
processors: [memory_limiter, batch, resourcedetection]
exporters: [signalfx]
# Use instead when sending to gateway
#exporters: [otlp]
#exporters: [otlp/gateway]
# Required for Splunk Infrastructure Monitoring
logs/signalfx:
receivers: [signalfx]
processors: [memory_limiter, batch]
exporters: [signalfx]
# Use instead when sending to gateway
#exporters: [otlp]
#exporters: [otlp/gateway]
logs/entities:
# Receivers are dynamically added if discovery mode is enabled
receivers: [nop]
Expand All @@ -201,4 +202,4 @@ service:
#- resource/add_environment
exporters: [splunk_hec/profiling]
# Use instead when sending to gateway
#exporters: [otlp]
#exporters: [otlp/gateway]
9 changes: 5 additions & 4 deletions cmd/otelcol/fips/config/agent_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,10 @@ processors:

exporters:
# Traces
sapm:
access_token: "${SPLUNK_ACCESS_TOKEN}"
endpoint: "${SPLUNK_INGEST_URL}/v2/trace"
otlphttp:
traces_endpoint: "${SPLUNK_INGEST_URL}/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
# Metrics + Events
signalfx:
access_token: "${SPLUNK_ACCESS_TOKEN}"
Expand Down Expand Up @@ -168,7 +169,7 @@ service:
- batch
- resourcedetection
#- resource/add_environment
exporters: [sapm, signalfx]
exporters: [otlphttp, signalfx]
# Use instead when sending to gateway
#exporters: [otlp, signalfx]
metrics:
Expand Down
4 changes: 4 additions & 0 deletions deployments/nomad/otel-agent.nomad
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,10 @@ exporters:
sync_host_metadata: true
debug:
verbosity: detailed
otlphttp:
traces_endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
service:
extensions:
- health_check
Expand Down
4 changes: 4 additions & 0 deletions deployments/nomad/otel-gateway.nomad
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,10 @@ exporters:
ingest_url: https://ingest.${SPLUNK_REALM}.signalfx.com
debug:
verbosity: detailed
otlphttp:
traces_endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"
service:
extensions:
- health_check
Expand Down
21 changes: 12 additions & 9 deletions internal/settings/settings_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -372,12 +372,18 @@ func TestSetDefaultEnvVarsSetsInterfaceFromConfigOption(t *testing.T) {
func TestSetDefaultFeatureGatesRespectsOverrides(t *testing.T) {
t.Cleanup(setRequiredEnvVars(t))
for _, args := range [][]string{
{"--feature-gates", "some-gate", "--feature-gates", "telemetry.useOtelForInternalMetrics", "--feature-gates",
"another-gate"},
{"--feature-gates", "some-gate", "--feature-gates", "+telemetry.useOtelForInternalMetrics",
"--feature-gates", "another-gate"},
{"--feature-gates", "some-gate", "--feature-gates", "-telemetry.useOtelForInternalMetrics",
"--feature-gates", "another-gate"},
{
"--feature-gates", "some-gate", "--feature-gates", "telemetry.useOtelForInternalMetrics", "--feature-gates",
"another-gate",
},
{
"--feature-gates", "some-gate", "--feature-gates", "+telemetry.useOtelForInternalMetrics",
"--feature-gates", "another-gate",
},
{
"--feature-gates", "some-gate", "--feature-gates", "-telemetry.useOtelForInternalMetrics",
"--feature-gates", "another-gate",
},
} {
t.Run(strings.Join(args, " "), func(t *testing.T) {
settings, err := New(args)
Expand All @@ -388,7 +394,6 @@ func TestSetDefaultFeatureGatesRespectsOverrides(t *testing.T) {
}

func TestSetSoftMemLimitWithoutGoMemLimitEnvVar(t *testing.T) {

// if GOLIMIT is not set, we expect soft limit to be 90% of the total memory env var or 90% of default total memory 512 Mib.
t.Cleanup(setRequiredEnvVars(t))
require.NoError(t, os.Setenv(MemTotalEnvVar, "200"))
Expand All @@ -402,7 +407,6 @@ func TestSetSoftMemLimitWithoutGoMemLimitEnvVar(t *testing.T) {
require.NoError(t, err)
require.NotNil(t, settings)
require.Equal(t, int64(482344960), debug.SetMemoryLimit(-1))

}

func TestUseConfigPathsFromEnvVar(t *testing.T) {
Expand Down Expand Up @@ -591,7 +595,6 @@ func TestConfigArgEnvURIForm(t *testing.T) {
require.NoError(t, err)
require.Equal(t, []string{"env:SOME_ENV_VAR"}, settings.configPaths.value)
require.Equal(t, settings.configPaths.value, settings.ResolverURIs())

}

func TestCheckRuntimeParams_MemTotal(t *testing.T) {
Expand Down
10 changes: 6 additions & 4 deletions pkg/receiver/smartagentreceiver/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,11 @@ exporters:
signalfx:
access_token: "${SIGNALFX_ACCESS_TOKEN}"
realm: us1
sapm:
access_token: "${SIGNALFX_ACCESS_TOKEN}"
endpoint: https://ingest.us1.signalfx.com/v2/trace
otlphttp:
metrics_endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/datapoint/otlp"
traces_endpoint: "https://ingest.${SPLUNK_REALM}.signalfx.com/v2/trace/otlp"
headers:
"X-SF-Token": "${SPLUNK_ACCESS_TOKEN}"

service:
pipelines:
Expand All @@ -98,5 +100,5 @@ service:
processors:
- resourcedetection
exporters:
- sapm
- otlphttp
```
Loading
Loading