Skip to content

Conversation

@evgeny-leksikov
Copy link
Contributor

What?

  • Add nixl_capi_get_xfer_telemetry C API function in wrapper.h/cpp
  • Add XferTelemetry struct with timing and performance metrics
  • Add get_xfer_telemetry method to Agent implementation
  • Add get_telemetry convenience method to XferRequest
  • Add comprehensive tests for telemetry functionality
  • Support both enabled/disabled telemetry scenarios
  • Include proper error handling for TelemetryNotEnabled cases

Why?

The telemetry API allows Rust users to retrieve transfer performance metrics including start time, post/transfer durations, total bytes, and descriptor count. Includes convenience methods for time conversion and transfer rate calculations.

@github-actions
Copy link

👋 Hi evgeny-leksikov! Thank you for contributing to ai-dynamo/nixl.

Your PR reviewers will review your contribution then trigger the CI to test your changes.

🚀

@evgeny-leksikov
Copy link
Contributor Author

/build

@ovidiusm
Copy link
Contributor

/build

@evgeny-leksikov
Copy link
Contributor Author

/build

gleon99
gleon99 previously approved these changes Sep 29, 2025
@gleon99 gleon99 enabled auto-merge (squash) September 29, 2025 15:14
@evgeny-leksikov
Copy link
Contributor Author

/build

@gleon99 gleon99 merged commit 9d89994 into ai-dynamo:main Sep 29, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants