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

[chore] Update event semantic conventions #880

Merged
merged 7 commits into from
May 3, 2024

Conversation

MSNev
Copy link
Contributor

@MSNev MSNev commented Apr 4, 2024

Provides additional context for #755

Changes

Provides additional clarification, structure and conventions for describing a Log based event.

@MSNev MSNev requested review from a team April 4, 2024 19:41
@MSNev MSNev force-pushed the MSNev/UpdateEventsDef branch from 649ffbd to 334298d Compare April 4, 2024 19:43
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
@trisch-me trisch-me mentioned this pull request Apr 8, 2024
3 tasks
docs/general/events.md Outdated Show resolved Hide resolved
@MSNev MSNev force-pushed the MSNev/UpdateEventsDef branch 2 times, most recently from 7b6aac3 to 462ac73 Compare April 11, 2024 00:13
@MSNev
Copy link
Contributor Author

MSNev commented Apr 11, 2024

link failure seems unrelated to this PR, can someone re-trigger the task to get it re-evaluated.

docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
docs/general/events.md Outdated Show resolved Hide resolved
@MSNev MSNev force-pushed the MSNev/UpdateEventsDef branch from 900bf5f to e51b43a Compare April 19, 2024 19:57
@MSNev
Copy link
Contributor Author

MSNev commented Apr 19, 2024

unrelated link check failure again :-(

docs/general/events.md Show resolved Hide resolved
@lmolkova lmolkova merged commit 55fb2c0 into open-telemetry:main May 3, 2024
11 of 12 checks passed
drewby pushed a commit to drewby/otel-semantic-conventions that referenced this pull request May 23, 2024
Co-authored-by: Joao Grassi <5938087+joaopgrassi@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Archived in project
Development

Successfully merging this pull request may close these issues.

7 participants