-
Notifications
You must be signed in to change notification settings - Fork 264
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 specification to include development profiles #582
Conversation
cc @open-telemetry/profiling-approvers @open-telemetry/profiling-maintainers @open-telemetry/profiling-triagers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just need to fix one round bracket.
Co-authored-by: Tigran Najaryan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks.
We're waiting for this change to move forward with the collector OTLP receiver. Could we get more reviews/a merge? |
As discussed in #582 (comment), this renames the `v1experimental` version for profiles to `v1development`. I've also taken this opportunity to update the versioning and stability link.
This updates the OTLP specification to have appropriate mentions of the profiles signal (and mention it's experimental).