Skip to content

implement NRI plugin reconnect mechanism#1763

Open
tariq1890 wants to merge 1 commit intomainfrom
nri-plugin-reconnect
Open

implement NRI plugin reconnect mechanism#1763
tariq1890 wants to merge 1 commit intomainfrom
nri-plugin-reconnect

Conversation

@tariq1890
Copy link
Copy Markdown
Contributor

When the container runtime (containerd/cri-o) is stopped for some reason, the plugin server started up the toolkit loses connection to the container runtime's TTRPC server and it is terminated as a result. For resiliency, it is critical that the NRI plugin reconnect to the NRI ttrpc server in the event of a container runtime restart and continue to be able to server management CDI device injection requests.

When the container runtime (containerd/cri-o) is stopped for some reason,
the plugin server started up the toolkit loses connection to the container
runtime's TTRPC server and it is terminated as a result. For resiliency,
it is critical that the NRI plugin reconnect to the NRI ttrpc server in
the event of a container runtime restart and continue to be able to server
management CDI device injection requests.

Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
@tariq1890 tariq1890 requested a review from cdesiniotis April 13, 2026 21:51
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 24368761080

Coverage decreased (-0.08%) to 43.321%

Details

  • Coverage decreased (-0.08%) from the base build.
  • Patch coverage: 28 uncovered changes across 1 file (0 of 28 lines covered, 0.0%).
  • 2 coverage regressions across 1 file.

Uncovered Changes

File Changed Covered %
cmd/nvidia-ctk-installer/container/runtime/nri/plugin.go 28 0 0.0%

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
cmd/nvidia-ctk-installer/container/runtime/nri/plugin.go 2 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 14852
Covered Lines: 6434
Line Coverage: 43.32%
Coverage Strength: 0.48 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants