Skip to content

Commit

Permalink
Remove TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
dricross committed Jan 8, 2025
1 parent 2bd8446 commit 2daedb1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
3 changes: 1 addition & 2 deletions exporter/awsemfexporter/emf_exporter.go
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ func newEmfExporter(config *Config, set exporter.Settings) (*emfExporter, error)
processResourceLabels: func(map[string]string) {},
}

// TODO(kausyas): Check why this isnt in upstream
if config.IsAppSignalsEnabled() {
userAgent := appsignals.NewUserAgent()
emfExporter.processResourceLabels = userAgent.Process
Expand All @@ -104,7 +103,7 @@ func (emf *emfExporter) pushMetricsData(_ context.Context, md pmetric.Metrics) e
}
}
emf.config.logger.Debug("Start processing resource metrics", zap.Any("labels", labels))
emf.processResourceLabels(labels) // TODO(kausyas): Check why this isnt in upstream
emf.processResourceLabels(labels)

groupedMetrics := make(map[any]*groupedMetric)
defaultLogStream := fmt.Sprintf("otel-stream-%s", emf.collectorID)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ func TestGetMetricKey(t *testing.T) {
ci.ContainerNamekey: "containerName",
},
}
// TODO(kausyas): Make sure this isnt exported anywhere
assert.Equal(t, "metricType:Container,podId:podID,containerName:containerName", getMetricKey(c))

c = &stores.CIMetricImpl{
Expand Down

0 comments on commit 2daedb1

Please sign in to comment.