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

Update status of stdout exporter for logs to stable #3901

Closed
wants to merge 1 commit into from

Conversation

cijothomas
Copy link
Member

https://github.com/open-telemetry/opentelemetry-specification/pull/3741/files and https://github.com/open-telemetry/opentelemetry-specification/pull/3740/files were added together, and it is not clear why one is experimental, and the other is stable.

This PR is marking the logging stdout exporter spec as stable. I couldn't find any reason why it was non-stable in the first place, so I assume it was just an oversight only. Happy to be corrected, if I missed something

Fixes #

Changes

Please provide a brief description of the changes here.

For non-trivial changes, follow the change proposal process.

https://github.com/open-telemetry/opentelemetry-specification/pull/3741/files and 
https://github.com/open-telemetry/opentelemetry-specification/pull/3740/files
were added together, and it is not clear why one is experimental, and the other is stable.

This PR is marking the logging stdout exporter spec as stable. I couldn't find any reason why it was non-stable in the first place, so I assume it was just an oversight only. Happy to be corrected, if I missed something
Copy link

github-actions bot commented Mar 4, 2024

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Mar 4, 2024
@carlosalberto
Copy link
Contributor

@open-telemetry/specs-logs-approvers please review

Also @cijothomas we usually, for PR purposes, don't create branches from this very repo, but from our personal ones, etc.

@cijothomas
Copy link
Member Author

@open-telemetry/specs-logs-approvers please review

Also @cijothomas we usually, for PR purposes, don't create branches from this very repo, but from our personal ones, etc.

that was my mistake! I used browser to edit, assuming it creates branch in my fork!

@cijothomas
Copy link
Member Author

closing in favor of #3922 as this was branched in this repo, not a fork.

@cijothomas cijothomas closed this Mar 5, 2024
@reyang reyang deleted the cijothomas-patch-1 branch March 5, 2024 19:58
@reyang
Copy link
Member

reyang commented Mar 5, 2024

I've cleaned up personal branches for this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants