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

Blog post regarding ECS and OTel SemConv Convergence #2594

Merged

Conversation

reyang
Copy link
Member

@reyang reyang commented Apr 12, 2023

Note: we would like the OTel blog published 7am Amsterdam time on April 18th which is 10pm PT on April 17th.

Background context

The stakeholders of ECS (Elastic Common Schema) and OpenTelemetry have been making progress since the past 12 months with the outcome of OTEP 199. Since the OTEP 199 got approved/merged, we've had multiple discussions among the ECS stakeholders from Elastic, OpenTelemetry GC (Governance Committee) and TC (Technical Committee) members, now we've reached to a point that we are ready to publicly announce this merge in the upcoming EU KubeCon event ❤️🍾🥳!

What's the communication plan

The plan is that Elastic and OpenTelemetry will each publish an official post/announcement at 7am Amsterdam time on April 18th which is 10pm PT on April 17th (right before KubeCon). This PR covers the blog post from OpenTelemetry, while Elastic will publish their announcement to https://elastic.co/blog/ecs-elastic-common-schema-otel-opentelemetry-announcement (note that the link will return 404 at this moment, and will become available at the publishing time).

Additional info

A draft Google Doc was prepared and went through several rounds of internal reviews in the past two weeks, which resulted in this PR after the alignment. The original doc can be found from https://docs.google.com/document/d/1ymr4sVG-Npdr1-hys-PKhF6pZXc_bNYmR1fee_n9GWM/edit?usp=sharing.

@AlexanderWert @open-telemetry/governance-committee @open-telemetry/technical-committee

@reyang reyang requested review from a team as code owners April 12, 2023 05:11
@reyang reyang changed the title Blog post regarding ECS and OTel SemConv Convergence [DO NOT MERGE] Blog post regarding ECS and OTel SemConv Convergence Apr 12, 2023
@chalin chalin force-pushed the reyang/ecs-otel-semconv-convergence branch from 6dd35e8 to 8aacfc9 Compare April 12, 2023 15:27
@chalin
Copy link
Contributor

chalin commented Apr 12, 2023

FYI, I rebased, ran the formatter and updated the refcache. I'll soon submit a few suggested copyedits.

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copyedits and a fix for the build warning. See inline comments.

content/en/blog/2023/ecs-otel-semconv-convergence.md Outdated Show resolved Hide resolved
content/en/blog/2023/ecs-otel-semconv-convergence.md Outdated Show resolved Hide resolved
Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should signal in this post that this means breaking changes are afoot (and they're significant). There's a lot of people who have been using http instrumentation that's been stable for a long time, and all of that is changing. It will hopefully be the case that all the telemetry backends can make this transition seamless for their users, but we should also call that out as a goal and that we're working with backend vendors to define that transition path.

@reyang
Copy link
Member Author

reyang commented Apr 13, 2023

a lot of people who have been using http instrumentation that's been stable for a long time, and all of that is changing

Where is the "stable for a long time" coming from?

It will hopefully be the case that all the telemetry backends can make this transition seamless for their users, but we should also call that out as a goal and that we're working with backend vendors to define that transition path.

This is covered by the OTEP 199.

@cartermp
Copy link
Contributor

cartermp commented Apr 13, 2023

HTTP instrumentions haven't changed nearly close to this extent in a long, long time. I've been able to query on http.status_code and http.method since...I'd like to say Fall of 20201?...and run those same queries. All of that is now changing. That's a massive breaking change. I don't think it's sufficient to just link to an OTEP. We can do it without doom and gloom, but I think that a change of this scale should be explicitly called out with a summarization of what the current thinking is (i.e., end-users don't need to act but telemetry backends do) from the very start.

@austinlparker
Copy link
Member

I'll take responsibility for merging/publishing this fyi.

@austinlparker
Copy link
Member

@cartermp FWIW I don't think we'll be able to make edits on this post as it seems that this language was hashed out w/Elastic. I do think our path forward here should be to publish a companion piece/explainer and link to it.

@alolita
Copy link
Member

alolita commented Apr 13, 2023

Can we also reference the other related PRs in the References section of the post.

For example,
open-telemetry/oteps#197 (which is the original proposal)

@cartermp cartermp dismissed their stale review April 13, 2023 19:34

language pre-decided upon

@cartermp
Copy link
Contributor

I do think our path forward here should be to publish a companion piece/explainer and link to it.

Okay, that's fair. I'll engage with that when we do it.

@reyang
Copy link
Member Author

reyang commented Apr 13, 2023

Can we also reference the other related PRs in the References section of the post.

For example, open-telemetry/oteps#197 (which is the original proposal)

@alolita Thanks for the suggestion! I've added more reference links, PTAL.

@reyang reyang changed the title [DO NOT MERGE] Blog post regarding ECS and OTel SemConv Convergence Blog post regarding ECS and OTel SemConv Convergence Apr 15, 2023
@austinlparker
Copy link
Member

Need to get checks green before merge but I'm OK with copy as-is.

@reyang
Copy link
Member Author

reyang commented Apr 17, 2023

Need to get checks green before merge but I'm OK with copy as-is.

I've turned all the CI to green.

@austinlparker austinlparker merged commit f54add0 into open-telemetry:main Apr 18, 2023
6 checks passed
@austinlparker
Copy link
Member

Goin' live!

@reyang reyang deleted the reyang/ecs-otel-semconv-convergence branch April 18, 2023 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants