feat(rdmacm): add wrapper for rdma_migrate_id

[dragonJACson]

Fix #112

[gemini-code-assist]

Code Review

This pull request introduces the ability to migrate an Identifier to a different EventChannel by wrapping the identifier's event channel in a Mutex and implementing the migrate method using rdma_migrate_id. It also adds corresponding error types, a test helper wait_for_cm_event, and unit tests. However, a critical lifetime and memory safety issue was identified: migrating an identifier can lead to a use-after-free or undefined behavior if there are unacknowledged events from the old event channel, as Event does not currently keep its source EventChannel alive. It is recommended to update the Event struct to hold an Arc<EventChannel> to prevent this.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[codecov]

Codecov Report

❌ Patch coverage is 94.78261% with 6 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/rdmacm/communication_manager.rs	94.78%	6 Missing ⚠️

Files with missing lines	Coverage Δ
src/rdmacm/communication_manager.rs	88.67% <94.78%> (+1.11%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Adds a safe Rust wrapper around rdma_migrate_id(3) in the rdmacm communication manager layer, enabling an Identifier (rdma_cm_id) to be migrated to a different EventChannel while keeping the underlying channel alive across migrations.

Changes:

• Introduced Identifier::migrate plus MigrateError/MigrateErrorKind to wrap rdma_migrate_id.
• Updated Identifier to hold its current EventChannel in a Mutex<Arc<EventChannel>> to keep lifetimes correct across migrations and serialize concurrent migrations.
• Added/updated tests with a small polling-based helper (wait_for_cm_event) and new test coverage for event delivery after migration.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.