Skip to content

feat: enable OTLP/HTTP for metrics#63

Merged
LucasAlvesSoares merged 5 commits intomainfrom
http-provider-for-metrics
Apr 20, 2026
Merged

feat: enable OTLP/HTTP for metrics#63
LucasAlvesSoares merged 5 commits intomainfrom
http-provider-for-metrics

Conversation

@LucasAlvesSoares
Copy link
Copy Markdown
Contributor

Description

Enabled configuring protocol for metering via the OTEL_EXPORTER_OTLP_PROTOCOL env var

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

How to Test

  1. Add environment variable OTEL_EXPORTER_OTLP_PROTOCOL="http/protobuf"
  2. Emit metrics by set_aicore_config() using SAP Cloud SDK
  3. Metrics will be exported using HTTP protocol

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

@LucasAlvesSoares LucasAlvesSoares marked this pull request as ready for review April 20, 2026 23:18
@LucasAlvesSoares LucasAlvesSoares requested a review from a team as a code owner April 20, 2026 23:18
@LucasAlvesSoares LucasAlvesSoares merged commit e9f529d into main Apr 20, 2026
9 of 10 checks passed
@LucasAlvesSoares LucasAlvesSoares deleted the http-provider-for-metrics branch April 20, 2026 23:25
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