Skip to content

Commit

Permalink
Revert "[pkg/otlp] Correctly set serializer exporter capabilities (#1…
Browse files Browse the repository at this point in the history
…8626)"

This reverts commit c72698b.
  • Loading branch information
songy23 authored Aug 8, 2023
1 parent 59ff4f6 commit 95218f2
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 15 deletions.
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -514,7 +514,7 @@ require (
go.etcd.io/etcd/client/v3 v3.6.0-alpha.0 // indirect
go.etcd.io/etcd/server/v3 v3.6.0-alpha.0.0.20220522111935-c3bc4116dcd1 // indirect
go.opencensus.io v0.24.0 // indirect
go.opentelemetry.io/collector/consumer v0.75.0
go.opentelemetry.io/collector/consumer v0.75.0 // indirect
go.opentelemetry.io/collector/featuregate v0.75.0 // indirect
go.opentelemetry.io/collector/semconv v0.81.0 // indirect
go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.40.0 // indirect
Expand Down
3 changes: 0 additions & 3 deletions pkg/otlp/internal/serializerexporter/factory.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import (

"github.com/open-telemetry/opentelemetry-collector-contrib/pkg/resourcetotelemetry"
"go.opentelemetry.io/collector/component"
"go.opentelemetry.io/collector/consumer"
exp "go.opentelemetry.io/collector/exporter"
"go.opentelemetry.io/collector/exporter/exporterhelper"

Expand Down Expand Up @@ -49,8 +48,6 @@ func (f *factory) createMetricExporter(ctx context.Context, params exp.CreateSet
exporter, err := exporterhelper.NewMetricsExporter(ctx, params, cfg, newExp.ConsumeMetrics,
exporterhelper.WithQueue(cfg.QueueSettings),
exporterhelper.WithTimeout(cfg.TimeoutSettings),
// the metrics remapping code mutates data
exporterhelper.WithCapabilities(consumer.Capabilities{MutatesData: true}),
)
if err != nil {
return nil, err
Expand Down
11 changes: 0 additions & 11 deletions releasenotes/notes/otel-panic-4cc6fdb17f7f062f.yaml

This file was deleted.

0 comments on commit 95218f2

Please sign in to comment.