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.

Sign up for GitHub

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

Jump to bottom

Remove the sentence of discouraging direct use of Logger.Emit method #4275

Open
XSAM opened this issue Oct 30, 2024 · 3 comments
Open

Remove the sentence of discouraging direct use of Logger.Emit method #4275

XSAM opened this issue Oct 30, 2024 · 3 comments
Labels
area:api Cross language API specification issue sig-issue A specific SIG should look into this before discussing at the spec spec:logs Related to the specification/logs directory

Comments

@XSAM
Copy link
Member

XSAM commented Oct 30, 2024

Based on https://github.com/open-telemetry/oteps/blob/main/text/0265-event-vision.md#api, we are going to remove the sentence discouraging direct use.

Originally posted by @XSAM in #4259 (comment)

OpenTelemetry SHOULD provide a (user-facing) Logs API that includes the capability to emit OpenTelemetry Events.
@XSAM XSAM mentioned this issue Oct 30, 2024
Merged
@pellared
Copy link
Member

pellared commented Oct 30, 2024 •
edited
Loading

Related to (possible duplicate of):

@pellared pellared added area:api Cross language API specification issue spec:logs Related to the specification/logs directory labels Oct 30, 2024
@jpkrohling jpkrohling added the triage:deciding:tc-inbox Needs attention from the TC in order to move forward label Nov 4, 2024
@jpkrohling
Copy link
Member

It's not clear whether this is something with enough TC buy-in, we are assigning this to as TC Inbox.

@trask
Copy link
Member

trask commented Nov 6, 2024

removing tc-inbox since this is a sig-issue

@trask trask added sig-issue A specific SIG should look into this before discussing at the spec and removed triage:deciding:tc-inbox Needs attention from the TC in order to move forward labels Nov 6, 2024
@pellared pellared moved this to Todo in Logs SIG Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:api Cross language API specification issue sig-issue A specific SIG should look into this before discussing at the spec spec:logs Related to the specification/logs directory
Projects
Logs SIG
Status: No status
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jpkrohling @trask @pellared @XSAM