-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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: remove deprecated exporter #35367
exporter/instana: remove deprecated exporter #35367
Conversation
@codeboten can you rebase your PR? |
9c1fff6
to
cb29a0c
Compare
needs another rebase |
# (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: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
subtext: | |
subtext: Use the `otlp` exporter instead as explained in IBM's [documentation](https://www.ibm.com/docs/en/instana-observability/current?topic=opentelemetry-sending-data-instana-backend). |
Not blocking: Just a thought, it may be helpful to share the path forward and documentation here as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah thanks for this @crobert-1, i'll update the changelog in a separate PR
This removes the exporter, it has been deprecated since March 2024 Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
cb29a0c
to
af16d47
Compare
This removes the exporter, it has been deprecated since March 2024 Fixes open-telemetry#34994 --------- Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
This removes the exporter, it has been deprecated since March 2024
Fixes #34994