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

Add OpenTelemetry-Compatible Telemetry Utility Function #26

Open
beeme1mr opened this issue Feb 13, 2025 · 0 comments
Open

Add OpenTelemetry-Compatible Telemetry Utility Function #26

beeme1mr opened this issue Feb 13, 2025 · 0 comments

Comments

@beeme1mr
Copy link
Member

Description

Add a telemetry utility function to the SDK that can be used within the finally stage of a hook to create OpenTelemetry-compatible telemetry signals. It must comply with the OpenTelemetry specification and meet the technical requirements, including those from Appendix D of the OpenFeature Specification and the OpenTelemetry Semantic Conventions for Feature Flags.

Requirements

  • Must comply with the OpenTelemetry (OTel) specification.
  • Must be included in the SDK.
  • Must not include any third-party dependencies.
  • Must adhere to the observability conformance requirements in Appendix D of the OpenFeature Specification.

Example implementation

A JavaScript implementation can found here.
This was referenced Mar 19, 2025
Closed
Closed
@brian-chebon brian-chebon mentioned this issue Mar 31, 2025
Merged
ABC2015 pushed a commit that referenced this issue Mar 31, 2025
@brian-chebon
chore: add evaluation details and OpenTelemetry to hooks (#51)
c638263 
## This PR

- Adds evaluation details parameter to the finally hook stage
- Implements OpenTelemetry-compatible telemetry utility functions

### Related Issues

- Fixes Add evaluation details to finally hook stage #27 
- Fixes Add OpenTelemetry-Compatible Telemetry Utility Function #26

Signed-off-by: Brian Chebon <[email protected]>
@brian-chebon brian-chebon mentioned this issue Apr 1, 2025
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 1, 2025
@brian-chebon @ABC2015
@beeme1mr @ubabar123 @openfeaturebot
chore: merge development to main (#52)
6b945f3 
## This PR

- Updates the main branch with the changes in development branch
- Adds evaluation details parameter to the finally hook stage
- Implements OpenTelemetry-compatible telemetry utility functions
- updates notification workflows to cover new branches

 ## Related Issues

- Fixes Add evaluation details to finally hook stage #27 
- Fixes Add OpenTelemetry-Compatible Telemetry Utility Function #26

---------

Signed-off-by: Jeremy Andrews <[email protected]>
Signed-off-by: Michael Beemer <[email protected]>
Signed-off-by: Usman <[email protected]>
Signed-off-by: ubabar123 <[email protected]>
Signed-off-by: Brian Chebon <[email protected]>
Signed-off-by: OpenFeature Bot <[email protected]>
Co-authored-by: Jeremy Andrews <[email protected]>
Co-authored-by: Michael Beemer <[email protected]>
Co-authored-by: ubabar123 <[email protected]>
Co-authored-by: ABC2015 <[email protected]>
Co-authored-by: OpenFeature Bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@beeme1mr