Skip to content

Conversation

@mamar123
Copy link
Contributor

@mamar123 mamar123 commented Jul 10, 2025

What?

Introduce a GTest-based infrastructure for testing backend plugins. This includes tests for both local and remote transfers, along with plugin-specific tests for the OBJ backend.

Why?

A unified and automated testing setup is essential to ensure correctness and stability across different plugin implementations. It also simplifies regression testing and future development.

How?

Built a modular GTest framework for backend validation, implemented core test cases for transfer functionality, and added dedicated tests for the OBJ plugin using environment-driven configurations.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Jul 10, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions
Copy link

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

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

🚀

@mamar123 mamar123 marked this pull request as draft July 10, 2025 13:47
@aranadive
Copy link
Contributor

@mamar123 @glevnv Does this supersede PR #338? Please make sure that gtest infra for testing plugins is in single PR.

@mamar123
Copy link
Contributor Author

/build

@mamar123
Copy link
Contributor Author

/ok to test c084c8c

@mamar123
Copy link
Contributor Author

/build

@mamar123
Copy link
Contributor Author

/ok to test 4e11918

@mamar123
Copy link
Contributor Author

/ok to test c83fdc0

@barneuman
Copy link
Contributor

/ok to test f97535f

@barneuman
Copy link
Contributor

/build

3 similar comments
@barneuman
Copy link
Contributor

/build

@barneuman
Copy link
Contributor

/build

@barneuman
Copy link
Contributor

/build

@mamar123 mamar123 requested a review from dpressle July 23, 2025 20:07
@w1ldptr
Copy link
Contributor

w1ldptr commented Jul 24, 2025

/build

@barneuman
Copy link
Contributor

/ok to test 1389d05

@barneuman
Copy link
Contributor

/build

@barneuman
Copy link
Contributor

/ok to test 9083986

@w1ldptr w1ldptr merged commit 9b92499 into ai-dynamo:main Jul 24, 2025
10 of 11 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.

5 participants