Skip to content

Upgrade OpenFeature and add new telemetry Hooks#2332

Merged
julianocosta89 merged 4 commits intoopen-telemetry:mainfrom
askpt:askpt/upgrade-ofeat
Jul 4, 2025
Merged

Upgrade OpenFeature and add new telemetry Hooks#2332
julianocosta89 merged 4 commits intoopen-telemetry:mainfrom
askpt:askpt/upgrade-ofeat

Conversation

@askpt
Copy link
Copy Markdown
Member

@askpt askpt commented Jul 4, 2025

Changes

This pull request updates the src/cart project to enhance observability and modernise OpenFeature integration. Key changes include replacing deprecated hooks, adding new instrumentation, and upgrading package versions.

Observability Enhancements:

  • Replaced TracingHook with MetricsHook and TraceEnricherHook to improve telemetry capabilities. The old Hooks library is now deprecated. (src/cart/src/Program.cs, src/cart/src/Program.csL46-R47)
  • Added OpenFeature instrumentation to the metrics pipeline for better visibility into feature flag usage. (src/cart/src/Program.cs, src/cart/src/Program.csR75)

OpenFeature Modernisation:

  • Updated OpenFeature package versions from 2.6.0 to 2.7.0 for OpenFeature, OpenFeature.DependencyInjection, and OpenFeature.Hosting. (src/cart/src/cart.csproj, src/cart/src/cart.csprojL33-R35)
  • Replaced the deprecated OpenFeature.Contrib.Hooks.Otel namespace with OpenFeature.Hooks to align with the latest library structure. (src/cart/src/Program.cs, src/cart/src/Program.csL20-R20)

Notes:

  • This PR brings the latest semantic conventions for feature flags to the OTEL demo project.

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

askpt added 4 commits July 4, 2025 07:35
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
…rHook

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
…try Hooks

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
@askpt askpt marked this pull request as ready for review July 4, 2025 07:08
@askpt askpt requested a review from a team as a code owner July 4, 2025 07:08
Copy link
Copy Markdown
Member

@julianocosta89 julianocosta89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks for the update!

@julianocosta89 julianocosta89 merged commit d64e452 into open-telemetry:main Jul 4, 2025
31 checks passed
@askpt askpt deleted the askpt/upgrade-ofeat branch July 4, 2025 07:52
rnjn pushed a commit to base-14/opentelemetry-demo that referenced this pull request Jul 28, 2025
* build(deps): update OpenFeature packages to version 2.7.0

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

* fix: update OpenFeature hooks to include MetricsHook and TraceEnricherHook

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

* feat: add OpenFeature meter to OpenTelemetry metrics configuration

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

* chore: update changelog to include OpenFeature upgrade and new telemetry Hooks

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

---------

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
cloud-hb pushed a commit to cloud-hb/opentelemetry-demo that referenced this pull request Nov 17, 2025
* build(deps): update OpenFeature packages to version 2.7.0

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

* fix: update OpenFeature hooks to include MetricsHook and TraceEnricherHook

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

* feat: add OpenFeature meter to OpenTelemetry metrics configuration

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

* chore: update changelog to include OpenFeature upgrade and new telemetry Hooks

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>

---------

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants