Skip to content

AGENT: cleanup metadata on failure - #509

Merged
yosefe merged 16 commits into
ai-dynamo:mainfrom
evgeny-leksikov:ucx_reconnect_after_failure
Aug 14, 2025
Merged

yosefe merged 16 commits into
ai-dynamo:mainfrom
evgeny-leksikov:ucx_reconnect_after_failure

Conversation

@evgeny-leksikov

@evgeny-leksikov evgeny-leksikov commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

What?

  1. clean up metadata on failure
  2. refactor load and invalidate remote data
  3. add reconnect case to error handling suite:
  • exchange metadata
  • data transfer (positive)
  • fail target
  • data transfer (negative)
  • re-init target and re-exchange metadata
  • data transfer

Why?

auto cleanup for reconnect

@github-actions

Copy link
Copy Markdown

👋 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.

🚀

Comment thread test/gtest/error_handling.cpp
Comment thread test/gtest/error_handling.cpp Outdated
Comment thread test/gtest/error_handling.cpp Outdated
Comment thread src/core/nixl_agent.cpp Outdated
Comment thread src/core/nixl_agent.cpp Outdated
Comment thread src/core/nixl_agent.cpp Outdated
Comment thread src/core/nixl_agent.cpp Outdated
Comment thread src/core/nixl_listener.cpp Outdated
Comment thread src/core/nixl_listener.cpp Outdated
Comment thread src/core/nixl_listener.cpp Outdated
Comment thread src/core/nixl_listener.cpp Outdated
Comment thread src/core/nixl_listener.cpp
Comment thread src/core/nixl_listener.cpp
Comment thread src/core/nixl_agent.cpp
@evgeny-leksikov

Copy link
Copy Markdown
Contributor Author

/build

Comment thread src/core/nixl_listener.cpp Outdated
Comment thread src/core/transfer_request.h Outdated
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
This reverts commit 93e7b47.

Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
Signed-off-by: Evgeny Leksikov <evgenylek@nvidia.com>
@evgeny-leksikov

Copy link
Copy Markdown
Contributor Author

/build

rakhmets
rakhmets previously approved these changes Aug 8, 2025
Comment thread src/core/nixl_agent.cpp
Comment thread test/gtest/error_handling.cpp Outdated
@roiedanino

Copy link
Copy Markdown
Contributor

/build

@evgeny-leksikov

Copy link
Copy Markdown
Contributor Author

/build

@yosefe

yosefe commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

/build

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.

5 participants