Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate the opentelemetry-collector chart #231

Open
TylerHelmuth opened this issue Apr 12, 2023 · 0 comments
Open

Deprecate the opentelemetry-collector chart #231

TylerHelmuth opened this issue Apr 12, 2023 · 0 comments
Labels
type: discussion Requests for comments, discussions about possible enhancements.

Comments

@TylerHelmuth
Copy link
Contributor

The Honeycomb opentelemetry-collector chart was created before a community opentelemetry-collector chart existed and filled the important need to easily deploy a collector that could export data to Honeycomb.

Today there is an OpenTelemetry Community Collector helm chart that is actively maintained and supports more features than our chart. In addition, exporting data to Honeycomb via the OpenTelemetry Collector has been simplified to the OTLP exporter with a small amount of configuration. At this time the only "advantage" our chart has is the ability to configure the OTLP exporter based on the user-supplied API key and dataset name. But the advantage comes at the cost of lack of useful k8s features, automatic complex component support (see the community chart's preset concept for more details`), and regular updates.

I propose we deprecate our opentelemetry-collector chart in favor of the community opentelemetry-collector chart, supplementing the Honeycomb-specific configuration that our chart provided with Honeycomb documentation. Instead of investing our time in our own chart, we invest our time helping maintain the OpenTelemetry community chart.

@TylerHelmuth TylerHelmuth added the type: discussion Requests for comments, discussions about possible enhancements. label Apr 12, 2023
TylerHelmuth added a commit that referenced this issue May 1, 2023
## Which problem is this PR solving?

- Related to #231

## Short description of the changes
- Updates the collector chart's readme with a Sunset notice.

---------

Co-authored-by: Phillip Carter <pcarter@fastmail.com>
TylerHelmuth added a commit that referenced this issue May 23, 2023
## Which problem is this PR solving?

- works towards #231

## Short description of the changes

- updates chart.yaml to deprecate the chart
- update readme to explicitly list deprecation
- update NOTES.txt to warn about deprecation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: discussion Requests for comments, discussions about possible enhancements.
Projects
None yet
Development

No branches or pull requests

1 participant