Skip to content

Commit

Permalink
chore(observability)!: remove deprecated component_name metric tag (v…
Browse files Browse the repository at this point in the history
…ectordotdev#18942)

* chore(datadog)!: remove deprecated config options

* update cue and add upgrade guide

* Update website/content/en/highlights/2023-11-07-0-34-0-upgrade-guide.md

Co-authored-by: Jesse Szwedko <jesse.szwedko@datadoghq.com>

* chore(observability)!: remove deprecated `component_name` metric tag

* remove remaining references

---------

Co-authored-by: Jesse Szwedko <jesse.szwedko@datadoghq.com>
  • Loading branch information
dsmith3197 and jszwedko committed Oct 26, 2023
1 parent e7b563d commit c6f5d2b
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 24 deletions.
1 change: 0 additions & 1 deletion lib/vector-core/src/metrics/label_filter.rs
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ impl LabelFilter for VectorLabelFilter {
key == "component_id"
|| key == "component_type"
|| key == "component_kind"
|| key == "component_name"
|| key == "buffer_type"
}
}
2 changes: 1 addition & 1 deletion src/config/source.rs
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ impl SourceContext {
if config.enabled() {
warn!(
message = "Enabling `acknowledgements` on sources themselves is deprecated in favor of enabling them in the sink configuration, and will be removed in a future version.",
component_name = self.key.id(),
component_id = self.key.id(),
);
}

Expand Down
10 changes: 0 additions & 10 deletions src/topology/builder.rs
Original file line number Diff line number Diff line change
Expand Up @@ -219,8 +219,6 @@ impl<'a> Builder<'a> {
component_kind = "source",
component_id = %key.id(),
component_type = %source.inner.get_component_name(),
// maintained for compatibility
component_name = %key.id(),
);
let _entered_span = span.enter();

Expand Down Expand Up @@ -428,8 +426,6 @@ impl<'a> Builder<'a> {
component_kind = "transform",
component_id = %key.id(),
component_type = %transform.inner.get_component_name(),
// maintained for compatibility
component_name = %key.id(),
);

// Create a map of the outputs to the list of possible definitions from those outputs.
Expand Down Expand Up @@ -514,8 +510,6 @@ impl<'a> Builder<'a> {
component_kind = "sink",
component_id = %key.id(),
component_type = %sink.inner.get_component_name(),
// maintained for compatibility
component_name = %key.id(),
);
let _entered_span = span.enter();

Expand Down Expand Up @@ -634,8 +628,6 @@ impl<'a> Builder<'a> {
component_kind = "sink",
component_type = typetag,
component_id = %component_key.id(),
// maintained for compatibility
component_name = %component_key.id(),
);
Err(TaskError::wrapped(error))
}
Expand All @@ -645,8 +637,6 @@ impl<'a> Builder<'a> {
component_kind = "sink",
component_type = typetag,
component_id = %component_key.id(),
// maintained for compatibility
component_name = %component_key.id(),
);
Err(TaskError::wrapped(Box::new(e)))
}
Expand Down
6 changes: 0 additions & 6 deletions src/topology/running.rs
Original file line number Diff line number Diff line change
Expand Up @@ -845,8 +845,6 @@ impl RunningTopology {
component_kind = "sink",
component_id = %task.id(),
component_type = %task.typetag(),
// maintained for compatibility
component_name = %task.id(),
);

let task_span = span.or_current();
Expand Down Expand Up @@ -888,8 +886,6 @@ impl RunningTopology {
component_kind = "transform",
component_id = %task.id(),
component_type = %task.typetag(),
// maintained for compatibility
component_name = %task.id(),
);

let task_span = span.or_current();
Expand Down Expand Up @@ -931,8 +927,6 @@ impl RunningTopology {
component_kind = "source",
component_id = %task.id(),
component_type = %task.typetag(),
// maintained for compatibility
component_name = %task.id(),
);

let task_span = span.or_current();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ badges:
Vector's 0.34.0 release includes **breaking changes**:

1. [Removal of Deprecated Datadog Component Config Options](#datadog-deprecated-config-options)
1. [Removal of Deprecated `component_name` Metric Tag](#deprecated-component-name)

We cover them below to help you upgrade quickly:

Expand All @@ -26,3 +27,7 @@ and Metrics sinks. Instead the `site` option should be used.

The `region` and `application_key` config options have
been removed from the Enterprise configuration. Instead of `region`, `site` should be used. `application_key` is no longer required.

#### Removal of Deprecated `component_name` Metric Tag {#deprecated-component-name}

The deprecated `component_name` tag has been removed from all internal metrics. Instead the `component_id` tag should be used.
6 changes: 0 additions & 6 deletions website/cue/reference/components/sources/internal_metrics.cue
Original file line number Diff line number Diff line change
Expand Up @@ -1094,7 +1094,6 @@ components: sources: internal_metrics: {
_component_tags: _internal_metrics_tags & {
component_kind: _component_kind
component_id: _component_id
component_name: _component_name
component_type: _component_type
}

Expand All @@ -1117,11 +1116,6 @@ components: sources: internal_metrics: {
required: true
examples: ["my_source", "my_sink"]
}
_component_name: {
description: "Deprecated, use `component_id` instead. The value is the same as `component_id`."
required: true
examples: ["my_source", "my_sink"]
}
_component_type: {
description: "The Vector component type."
required: true
Expand Down

0 comments on commit c6f5d2b

Please sign in to comment.