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

[exporter/instana] Consider deprecating component #31765

Closed
crobert-1 opened this issue Mar 14, 2024 · 6 comments · Fixed by #31821
Closed

[exporter/instana] Consider deprecating component #31765

crobert-1 opened this issue Mar 14, 2024 · 6 comments · Fixed by #31821

Comments

@crobert-1
Copy link
Member

Component(s)

exporter/instana

Describe the issue you're reporting

I bumped into this while reading #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.

@crobert-1 crobert-1 added the needs triage New item requiring triage label Mar 14, 2024
Copy link
Contributor

Pinging code owners:

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

@jpkrohling
Copy link
Member

jpkrohling commented Mar 15, 2024

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?

@jpkrohling jpkrohling removed the needs triage New item requiring triage label Mar 15, 2024
@hickeyma
Copy link
Contributor

@joshleecreates Do you mind answering this?

@joshleecreates
Copy link

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

@crobert-1
Copy link
Member Author

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 (#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
Copy link

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

dmitryax pushed a commit that referenced this issue Mar 26, 2024
**Description:**
Official Instana documentation recommends using the OTLP exporter with
specific configuration options, and already states this exporter is
deprecated.

**Link to tracking Issue:** Resolves #31765
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this issue May 8, 2024
…metry#31821)

**Description:**
Official Instana documentation recommends using the OTLP exporter with
specific configuration options, and already states this exporter is
deprecated.

**Link to tracking Issue:** Resolves open-telemetry#31765
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants