Add blog entry about Jaeger OTLP and deprecation by breedx-splk · Pull Request #1892 · open-telemetry/opentelemetry.io

breedx-splk · 2022-10-21T18:31:08Z

Feedback welcome. Thanks!

Preview: https://deploy-preview-1892--opentelemetry.netlify.app/blog/2022/jaeger-native-otlp/

reese-lee · 2022-10-21T19:07:11Z

Not a blog approver, but I think it's very clear and well-organized! I always appreciate these aspects of any written communication.

cartermp

Looks good overall! Just some small questions.

cartermp · 2022-10-22T00:38:07Z

+- Are you an OpenTelemetry user currently sending data to Jaeger?
+- Are you using the Jaeger client exporters, or have you already made the switch
+  to OTLP?
+- How painful will it be for you to migrate to native OTLP with Jaeger?


Not sure if this is worth putting in the blog post, but a fairly "obvious" action we should take is ensuring that every piece of docs that explains how to set up a jaeger exporter needs updating to show how to do it with OTLP. I would expect anyone's feedback to this question to be dependent on what the plan is for that.

Yeah agreed. I hope this isn't too much cart before the horse, and I would expect that we could update the docs during the deprecation phase.

svrnm

some comments

austinlparker

overall lgtm -- will this be cross-published onto the Jaeger blog?

chalin

See inline comments, thanks.

cartermp · 2022-10-28T12:56:11Z

@breedx-splk do you have permissions to cross-post to the jaeger blog as well?

Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>

breedx-splk · 2022-10-31T16:01:08Z

@breedx-splk do you have permissions to cross-post to the jaeger blog as well?
overall lgtm -- will this be cross-published onto the Jaeger blog?

@austinlparker @cartermp A really good idea. I've reached out to @yurishkuro on the spec issue to see if he can help with cross-posting.

cartermp · 2022-10-31T18:48:03Z

@breedx-splk sounds good to me. I am happy to merge as-is for now. Just a question of timing.

yurishkuro · 2022-10-31T20:06:32Z

I have a couple suggestions:

be explicit in the blog about which exporters are in scope, like Thrift over UDP, Thrift over HTTP, etc.
it may be worth a brief discussion on the implications of removing the exporters. Currently the blog sounds like it's nearly a complete no-op, just swap jaeger->otlp exporter, but that's not the case if people are running Jaeger agent.
could we add a poll to the PR asking people to vote for the preferred length of deprecation phase?

…he impact nature

breedx-splk · 2022-10-31T23:01:04Z

Ok, the survey is up and I've added links to it. I've also taken @yurishkuro's advice and spelled out exactly which exporters are to be deprecated and some high level descriptions of the user impact for each.
Thanks for helping out with this y'all.

Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>

govindrai · 2022-11-02T16:20:21Z

Now that Jaeger supports OTLP, this feels like a step backwards: It results in
an increased maintenance burden with very little benefit.

this text occurs is in the readme twice.

breedx-splk · 2022-11-02T16:24:28Z

Now that Jaeger supports OTLP, this feels like a step backwards: It results in
an increased maintenance burden with very little benefit.

this text occurs is in the readme twice.

Oof, thanks, I think that was a merge/rebase fail. Fixing that up now.

yurishkuro · 2022-11-03T17:50:31Z

@breedx-splk if you want to repost this to Jaeger blog, we have two options:

I can do it as a copy & paste with a back link, or
You can author the post on Medium and add it to the Jaeger publication. I will need to know your medium username to be able to add you as a writer.

breedx-splk · 2022-11-03T17:53:08Z

@breedx-splk if you want to repost this to Jaeger blog, we have two options:

I can do it as a copy & paste with a back link, or

You can author the post on Medium and add it to the Jaeger publication. I will need to know your medium username to be able to add you as a writer.

Thanks @yurishkuro. I'm flexible -- but I don't currently have a Medium account, so if you're willing to do a copy/paste that would be awesome! ❤️

yurishkuro · 2022-11-03T18:03:40Z

+
+We intend to **deprecate Jaeger exporters from OpenTelemetry** in the near
+future, and are looking for your feedback to determine the length of the
+depreation phase. The best way to provide feedback is by


typo: depreation -> deprecation

yurishkuro · 2022-11-03T18:13:27Z

https://medium.com/@YuriShkuro/better-alignment-with-opentelemetry-by-focusing-on-otlp-f3688939073f

breedx-splk · 2022-11-03T19:47:27Z

https://medium.com/@YuriShkuro/better-alignment-with-opentelemetry-by-focusing-on-otlp-f3688939073f

Thanks again for your help and support! 🤜🏻 🤛🏻

yurishkuro · 2022-11-03T20:10:39Z

Looking forward to the survey results. Please post them in the PR. Do you have something in mind for how long you want to keep the survey open? 1-2w?

breedx-splk · 2022-11-03T20:32:01Z

@yurishkuro 2w seems good to me.

add jaeger otlp and deprecation blog

f157584

breedx-splk requested review from a team October 21, 2022 18:31

cartermp approved these changes Oct 22, 2022

View reviewed changes

svrnm requested changes Oct 24, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md

address review comments

1cf8b3a

svrnm approved these changes Oct 28, 2022

View reviewed changes

austinlparker approved these changes Oct 28, 2022

View reviewed changes

chalin reviewed Oct 28, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

chalin reviewed Oct 28, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

chalin reviewed Oct 28, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

chalin reviewed Oct 28, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

breedx-splk and others added 2 commits October 31, 2022 08:50

Update content/en/blog/2022/jaeger-native-otlp/index.md

baa0816

Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>

Update content/en/blog/2022/jaeger-native-otlp/index.md

e06b7fe

Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>

This was referenced Oct 31, 2022

Deprecate jaeger exporters. open-telemetry/opentelemetry-specification#2858

Merged

Drop support for jaeger-thrift open-telemetry/opentelemetry-specification#2859

Closed

breedx-splk and others added 3 commits October 31, 2022 08:57

Update content/en/blog/2022/jaeger-native-otlp/index.md

35ee5c1

Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>

Update content/en/blog/2022/jaeger-native-otlp/index.md

f1bdf13

Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>

update date

3325059

chalin approved these changes Oct 31, 2022

View reviewed changes

breedx-splk added 4 commits October 31, 2022 15:27

add survey to blog post

41c0538

add explicit list of exporters and provide additional details about t…

401a653

…he impact nature

remove clarification clause

4ca5462

pretty

7d11f2a

chalin approved these changes Oct 31, 2022

View reviewed changes

yurishkuro approved these changes Nov 1, 2022

View reviewed changes

trask approved these changes Nov 1, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md

breedx-splk added 2 commits November 1, 2022 08:46

update date

357a3b5

remove question list in favor of just survey link

df07087

yurishkuro reviewed Nov 1, 2022

View reviewed changes

Comment thread content/en/blog/2022/jaeger-native-otlp/index.md Outdated

Update content/en/blog/2022/jaeger-native-otlp/index.md

91e8810

Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>

Kielek approved these changes Nov 2, 2022

View reviewed changes

bump date (again)

8bafbea

breedx-splk added 2 commits November 2, 2022 09:25

remove dupe sentence (merge failure)

5010cd8

update date

86c2e40

cartermp merged commit 7148ac6 into open-telemetry:main Nov 3, 2022

yurishkuro reviewed Nov 3, 2022

View reviewed changes

Conversation

breedx-splk commented Oct 21, 2022 • edited by chalin Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

reese-lee commented Oct 21, 2022

Uh oh!

cartermp left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cartermp Oct 22, 2022

Choose a reason for hiding this comment

Uh oh!

breedx-splk Oct 24, 2022

Choose a reason for hiding this comment

Uh oh!

svrnm left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

austinlparker left a comment

Choose a reason for hiding this comment

Uh oh!

chalin left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cartermp commented Oct 28, 2022

Uh oh!

Uh oh!

Uh oh!

breedx-splk commented Oct 31, 2022

Uh oh!

cartermp commented Oct 31, 2022

Uh oh!

yurishkuro commented Oct 31, 2022

Uh oh!

breedx-splk commented Oct 31, 2022

Uh oh!

Uh oh!

Uh oh!

Uh oh!

govindrai commented Nov 2, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

breedx-splk commented Nov 2, 2022

Uh oh!

yurishkuro commented Nov 3, 2022

Uh oh!

breedx-splk commented Nov 3, 2022

Uh oh!

yurishkuro Nov 3, 2022

Choose a reason for hiding this comment

Uh oh!

yurishkuro commented Nov 3, 2022

Uh oh!

breedx-splk commented Nov 3, 2022

Uh oh!

yurishkuro commented Nov 3, 2022

Uh oh!

breedx-splk commented Nov 3, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

breedx-splk commented Oct 21, 2022 •

edited by chalin

Loading

govindrai commented Nov 2, 2022 •

edited

Loading