Skip to content

Commit 21a9eed

Browse files
authored
Merge branch 'main' into feature/use-plain-command
2 parents 152be40 + a36c5ba commit 21a9eed

File tree

5 files changed

+9
-10
lines changed

5 files changed

+9
-10
lines changed

internal/beater/beater.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,12 @@ type RunnerParams struct {
124124
func NewRunner(args RunnerParams) (*Runner, error) {
125125
fips140.CheckFips()
126126

127+
// the default tracer is leaking and its background
128+
// goroutine is spamming requests to this apm server (default endpoint)
129+
// If TLS is enabled it causes "http request sent to https endpoint".
130+
// Close the default tracer since it's not used.
131+
apm.DefaultTracer().Close()
132+
127133
var unpackedConfig struct {
128134
APMServer *agentconfig.C `config:"apm-server"`
129135
Output agentconfig.Namespace `config:"output"`

testing/infra/k8s/base/stack/apm-server.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ metadata:
44
name: apm-server
55
spec:
66
# TODO: replace with a pinned version such as 9.0.0-aaaaaaaa-SNAPSHOT
7-
version: 9.1.0-f894ad4a-SNAPSHOT
7+
version: 9.1.0-2a2b7495-SNAPSHOT
88
count: 1
99
http:
1010
tls:

testing/infra/k8s/base/stack/elasticsearch.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ metadata:
44
name: elasticsearch
55
spec:
66
# TODO: replace with a pinned version such as 9.0.0-aaaaaaaa-SNAPSHOT
7-
version: 9.1.0-f894ad4a-SNAPSHOT
7+
version: 9.1.0-2a2b7495-SNAPSHOT
88
auth:
99
fileRealm:
1010
- secretName: elasticsearch-admin

testing/infra/k8s/base/stack/kibana.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ metadata:
44
name: kibana
55
spec:
66
# TODO: replace with a pinned version such as 9.0.0-aaaaaaaa-SNAPSHOT
7-
version: 9.1.0-f894ad4a-SNAPSHOT
7+
version: 9.1.0-2a2b7495-SNAPSHOT
88
count: 1
99
elasticsearchRef:
1010
name: elasticsearch

x-pack/apm-server/sampling/eventstorage/storage_manager.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -140,8 +140,6 @@ type StorageManager struct {
140140
// meterProvider is the OTel meter provider
141141
meterProvider metric.MeterProvider
142142
storageMetrics storageMetrics
143-
144-
metricRegistration metric.Registration
145143
}
146144

147145
type storageMetrics struct {
@@ -335,11 +333,6 @@ func (sm *StorageManager) Close() error {
335333
}
336334

337335
func (sm *StorageManager) close() error {
338-
if sm.metricRegistration != nil {
339-
if err := sm.metricRegistration.Unregister(); err != nil {
340-
sm.logger.With(logp.Error(err)).Error("failed to unregister metric")
341-
}
342-
}
343336
return errors.Join(
344337
wrapNonNilErr("event db flush error: %w", sm.eventDB.Flush()),
345338
wrapNonNilErr("decision db flush error: %w", sm.decisionDB.Flush()),

0 commit comments

Comments
 (0)