Skip to content

Commit

Permalink
[exporter/f5] deprecate exporter (#31186)
Browse files Browse the repository at this point in the history
After discussion with @lquerel, he mentioned the component is no longer
maintained at F5. Marking it as deprecated.

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
  • Loading branch information
Alex Boten authored Feb 12, 2024
1 parent ffa8b67 commit 020d744
Show file tree
Hide file tree
Showing 12 changed files with 40 additions and 20 deletions.
27 changes: 27 additions & 0 deletions .chloggen/codeboten_deprecate-f5exporter.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: deprecation

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: f5cloudexporter

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: deprecating component that is no longer maintained

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [31186]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:

# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: []
3 changes: 0 additions & 3 deletions cmd/configschema/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -480,7 +480,6 @@ require (
github.com/open-telemetry/opentelemetry-collector-contrib/connector/spanmetricsconnector v0.94.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/awskinesisexporter v0.94.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/datadogexporter v0.94.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter v0.94.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter v0.94.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudpubsubexporter v0.94.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlemanagedprometheusexporter v0.94.0 // indirect
Expand Down Expand Up @@ -780,8 +779,6 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/dynat

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/elasticsearchexporter => ../../exporter/elasticsearchexporter

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter => ../../exporter/f5cloudexporter

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/fileexporter => ../../exporter/fileexporter

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter => ../../exporter/googlecloudexporter
Expand Down
1 change: 0 additions & 1 deletion cmd/otelcontribcol/builder-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ exporters:
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/datasetexporter v0.94.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/dynatraceexporter v0.94.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/elasticsearchexporter v0.94.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter v0.94.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/fileexporter v0.94.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter v0.94.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudpubsubexporter v0.94.0
Expand Down
2 changes: 0 additions & 2 deletions cmd/otelcontribcol/components.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion cmd/otelcontribcol/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ require (
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/datasetexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/dynatraceexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/elasticsearchexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter v0.0.0-00010101000000-000000000000
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/fileexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudpubsubexporter v0.94.0
Expand Down
7 changes: 4 additions & 3 deletions exporter/f5cloudexporter/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,13 @@
<!-- status autogenerated section -->
| Status | |
| ------------- |-----------|
| Stability | [beta]: traces, metrics, logs |
| Stability | [deprecated]: traces, metrics, logs |
| Distributions | [contrib], [observiq] |
| Issues | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aexporter%2Ff5cloud%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Aexporter%2Ff5cloud) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aexporter%2Ff5cloud%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Aexporter%2Ff5cloud) |
| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@gramidt](https://www.github.com/gramidt) |
| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | |
| Emeritus | [@gramidt](https://www.github.com/gramidt) |

[beta]: https://github.com/open-telemetry/opentelemetry-collector#beta
[deprecated]: https://github.com/open-telemetry/opentelemetry-collector#deprecated
[contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib
[observiq]: https://github.com/observIQ/observiq-otel-collector
<!-- end autogenerated section -->
Expand Down
1 change: 1 addition & 0 deletions exporter/f5cloudexporter/doc.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,6 @@

//go:generate mdatagen metadata.yaml

// Deprecated: [v0.95.0] This package is deprecated and will be removed in a future release.
// Package f5cloudexporter implements an OpenTelemetry Collector exporter that sends trace, metric, and log data to F5 Cloud.
package f5cloudexporter // import "github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter"
1 change: 1 addition & 0 deletions exporter/f5cloudexporter/go.mod
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// Deprecated: [v0.95.0] This package is deprecated and will be removed in a future release.
module github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter

go 1.20
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 3 additions & 2 deletions exporter/f5cloudexporter/metadata.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ type: f5cloud
status:
class: exporter
stability:
beta: [traces, metrics, logs]
deprecated: [traces, metrics, logs]
distributions: [contrib, observiq]
codeowners:
active: [gramidt]
active: []
emeritus: [gramidt]
3 changes: 0 additions & 3 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ require (
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/datasetexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/dynatraceexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/elasticsearchexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/fileexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter v0.94.0
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudpubsubexporter v0.94.0
Expand Down Expand Up @@ -778,8 +777,6 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/datas

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/dynatraceexporter => ./exporter/dynatraceexporter

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter => ./exporter/f5cloudexporter

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/fileexporter => ./exporter/fileexporter

replace github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter => ./exporter/googlecloudexporter
Expand Down
2 changes: 0 additions & 2 deletions internal/components/components.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ import (
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/datasetexporter"
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/dynatraceexporter"
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/elasticsearchexporter"
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/f5cloudexporter"
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/fileexporter"
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudexporter"
"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudpubsubexporter"
Expand Down Expand Up @@ -328,7 +327,6 @@ func Components() (otelcol.Factories, error) {
debugexporter.NewFactory(),
dynatraceexporter.NewFactory(),
elasticsearchexporter.NewFactory(),
f5cloudexporter.NewFactory(),
fileexporter.NewFactory(),
googlecloudexporter.NewFactory(),
googlemanagedprometheusexporter.NewFactory(),
Expand Down

0 comments on commit 020d744

Please sign in to comment.