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

Remove deprecated Events API and SDK #4353

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

pellared
Copy link
Member

@pellared pellared commented Jan 8, 2025

@pellared pellared requested review from a team as code owners January 8, 2025 14:06
@pellared pellared self-assigned this Jan 8, 2025
@reyang reyang merged commit 9fbb758 into open-telemetry:main Jan 8, 2025
6 checks passed
@pellared pellared deleted the remove-events-api-sdk branch January 8, 2025 16:42
@trask
Copy link
Member

trask commented Jan 10, 2025

@open-telemetry/docs-approvers do we need to do anything about website related to the removal of these two pages?

@chalin
Copy link
Contributor

chalin commented Jan 10, 2025

Thanks for asking @trask. Yes, add two (path-local) aliases for the deleted pages to

aliases: [/docs/reference/specification/logs/overview]

@chalin
Copy link
Contributor

chalin commented Jan 10, 2025

Created an issue to track this:

@carlosalberto carlosalberto mentioned this pull request Jan 13, 2025
lmolkova pushed a commit to lmolkova/opentelemetry-specification that referenced this pull request Jan 14, 2025
carlosalberto added a commit that referenced this pull request Jan 22, 2025
### Logs

- Remove the deprecated Events API and SDK in favor of having Events
support in the Logs API and SDK.

([#4353](#4353))
- Remove `Logger`'s Log Instrumentation operations.

([#4352](#4352))
- Make all `Logger` operations user-facing.

([#4352](#4352))

### SDK Configuration

- Clarify that implementations should interpret timeout environment
variable values of zero as no limit (infinity).

([#4331](#4331))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants