Skip to content

Conversation

@ovidiusm
Copy link
Contributor

What?

Python wheel does not bundle libfabric dependencies. As a result, we get log errors on non-AWS systems:

+ python /workspace/examples/python/nixl_api_example.py
2025-10-28 10:54:56 NIXL INFO    nixl_api_example.py:35 Using NIXL Plugins from:
default location
E1028 10:54:56.298455       1 nixl_plugin_manager.cpp:122] Failed to load plugin from /workspace/venv/lib/python3.12/site-packages/nixl_cu12/../.nixl_cu12.mesonpy.libs/plugins/libplugin_LIBFABRIC.so: /opt/amazon/efa/lib/libfabric.so.1: version `FABRIC_1.8' not found (required by /workspace/venv/lib/python3.12/site-packages/nixl_cu12/../.nixl_cu12.mesonpy.libs/plugins/libplugin_LIBFABRIC.so)
E1028 10:54:56.298478       1 nixl_plugin_manager.cpp:288] Failed to load plugin 'LIBFABRIC' from any directory

These logs should be INFO.

…l dependencies

Signed-off-by: Ovidiu Mara <ovidium@nvidia.com>
@github-actions
Copy link

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

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

🚀

@ovidiusm
Copy link
Contributor Author

/build

@ovidiusm ovidiusm enabled auto-merge (squash) October 29, 2025 11:40
@brminich
Copy link
Contributor

/build

@ovidiusm ovidiusm merged commit b21955c into ai-dynamo:main Oct 29, 2025
20 of 21 checks passed
ovidiusm added a commit to ovidiusm/nixl that referenced this pull request Oct 29, 2025
…l dependencies (ai-dynamo#964)

Signed-off-by: Ovidiu Mara <ovidium@nvidia.com>
@ovidiusm ovidiusm deleted the fix-plugin-mgr-log branch October 29, 2025 15:32
nv-nmailhot pushed a commit that referenced this pull request Oct 31, 2025
…l dependencies (#964) (#967)

Signed-off-by: Ovidiu Mara <ovidium@nvidia.com>
e-ago pushed a commit to e-ago/nixl-doca-31 that referenced this pull request Nov 12, 2025
…l dependencies (ai-dynamo#964)

Signed-off-by: Ovidiu Mara <ovidium@nvidia.com>
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.

2 participants