Skip to content

Commit

Permalink
[chore] [exporter/signalfx] Remove redundant translation (open-teleme…
Browse files Browse the repository at this point in the history
…try#27634)

Remove translation dropping the `opencensus.resourcetype` resource
attribute as it's not emitted by k8s receivers anymore. The last one was
removed in
open-telemetry#27616
  • Loading branch information
dmitryax authored and JaredTan95 committed Oct 18, 2023
1 parent 0eecc7d commit 0981aa3
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions exporter/signalfxexporter/internal/translation/constants.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,6 @@ package translation // import "github.com/open-telemetry/opentelemetry-collector
// Keep it in YAML format to be able to easily copy and paste it in config if modifications needed.
const DefaultTranslationRulesYaml = `
translation_rules:
# drops opencensus.resourcetype dimension from metrics generated by receivers written
# using OC data structures. This rule can be removed once the k8s_cluster and kubeletstats
# receivers have been refactored to use pmetric.Metrics. These dimensions are added as a
# result of the conversion here https://github.com/open-telemetry/opentelemetry-collector/blob/v0.22.0/translator/internaldata/oc_to_resource.go#L128.
# Dropping these dimensions will ensure MTSes aren't broken when the receivers are
# refactored and this resource type dimension will cease to exist.
- action: drop_dimensions
metric_name: /^(k8s\.|container\.).*/
dimension_pairs:
opencensus.resourcetype:
k8s: true
container: true
- action: copy_metrics
mapping:
# kubeletstats container cpu needed for calculation below
Expand Down

0 comments on commit 0981aa3

Please sign in to comment.