Skip to content

Conversation

@nnshah1
Copy link
Contributor

@nnshah1 nnshah1 commented Sep 2, 2025

Overview:

Adds fault injection test cases for K8s Dynamo Deployments.

Details:

Adds a new managed_deployment class that enables the user to simplify launching deployments and injecting faults to test resilience..

Where should the reviewer start?

Readme.md
managed_deployment.py
tests/fault_tolerance/deploy/test_deployment.py

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 2, 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.

@nnshah1 nnshah1 marked this pull request as ready for review September 8, 2025 20:47
@nnshah1 nnshah1 requested review from a team as code owners September 8, 2025 20:47
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 8, 2025

Caution

Review failed

An error occurred during the review process. Please try again later.

Warning

Tools execution failed with the following error:

Failed to run tools: 8 RESOURCE_EXHAUSTED: Received message larger than max (6676063 vs 4194304)


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

paulhendricks and others added 17 commits September 8, 2025 16:06
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: Harry Kim <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: Julien Mancuso <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
Signed-off-by: Anant Sharma <[email protected]>
Signed-off-by: nnshah1 <[email protected]>
@nnshah1
Copy link
Contributor Author

nnshah1 commented Sep 8, 2025

Replaced with this #2942

@nnshah1 nnshah1 closed this Sep 8, 2025
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.