Skip to content

feat: grab SDK version from cmake#921

Merged
gsstoykov merged 4 commits intomainfrom
metadata-poc
Jun 4, 2025
Merged

feat: grab SDK version from cmake#921
gsstoykov merged 4 commits intomainfrom
metadata-poc

Conversation

@rwalworth
Copy link
Copy Markdown
Contributor

@rwalworth rwalworth commented Mar 17, 2025

Description:

Adds SDK name and version to the GRPC context with the goal of being intercepted for metrics.

Related issue(s):

Fixes #950

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.) - partially

Signed-off-by: Rob Walworth <robert.walworth@swirldslabs.com>
@rwalworth rwalworth mentioned this pull request May 13, 2025
gsstoykov added 3 commits June 4, 2025 11:03
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
@gsstoykov gsstoykov marked this pull request as ready for review June 4, 2025 08:58
@gsstoykov gsstoykov requested review from a team as code owners June 4, 2025 08:58
@gsstoykov
Copy link
Copy Markdown
Contributor

Fixed the issue with the agent. Local-node accepted the metadata so merging.

Copy link
Copy Markdown
Contributor

@gsstoykov gsstoykov left a comment

Choose a reason for hiding this comment

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

LGTM.

@gsstoykov gsstoykov merged commit d2ce183 into main Jun 4, 2025
10 checks passed
@gsstoykov gsstoykov deleted the metadata-poc branch June 4, 2025 10:03
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.

SDK metrics

2 participants