open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
2.92k stars 2.28k forks source link

[exporter/instana] Consider deprecating component #31765

Closed crobert-1 closed 6 months ago

crobert-1 commented 6 months ago

Component(s)

exporter/instana

Describe the issue you're reporting

I bumped into this while reading https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/30048, but for context, IBM Instana Observability is a vendor-specific backend that accepts trace data.

According to their documentation, this exporter is deprecated for usage. The officially recommended way to use the OpenTelemetry Collector to export telemetry is to use the OTLP exporter, and set a specific setting in your Instana backend.

Is there a reason to keep this exporter? Note that marking it as deprecated is not deleting it, but simply a notification that users should migrate to what Instana officially supports. Read here for deprecation information in the collector.

github-actions[bot] commented 6 months ago

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

jpkrohling commented 6 months ago

Sounds good to me, but given this is owned by a vendor, I'd request their input on this. @hickeyma, would you please comment on this?

hickeyma commented 6 months ago

@joshleecreates Do you mind answering this?

joshleecreates commented 6 months ago

Yes, we should mark this as deprecated, thank you!

crobert-1 commented 6 months ago

Thanks @hickeyma and @joshleecreates! Sorry about posting this out of the blue, it was meant more to be a question than anything else. I definitely didn't mean to push anything one way or another.

I've opened a PR (https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31821) that makes this change, but it's no problem to keep the exporter in this repository for as long as you need. The goal is to accurately reflect documentation and status here. 👍

joshleecreates commented 6 months ago

No problem at all, thank you for catching the discrepancy!