Skip to content

agent/csi/plugin, manager/csi: un-export nodePlugin.Client - #3278

Open
thaJeztah wants to merge 2 commits into
moby:masterfrom
thaJeztah:csi_unexport
Open

agent/csi/plugin, manager/csi: un-export nodePlugin.Client#3278
thaJeztah wants to merge 2 commits into
moby:masterfrom
thaJeztah:csi_unexport

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Aug 1, 2026

Copy link
Copy Markdown
Member

It's not part of a public interface; un-export it to make it clear that it's only for internal use.

- How to test it

- Description for the changelog

It's not part of a public interface; un-export it to make it clear that
it's only for internal use.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
It's not part of a public interface; un-export it to make it clear that
it's only for internal use.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah changed the title agent/csi/plugin: un-export nodePlugin.Client agent/csi/plugin, manager/csi: un-export nodePlugin.Client Aug 1, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 14.71%. Comparing base (6e9e7b8) to head (33205ef).
⚠️ Report is 60 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3278      +/-   ##
==========================================
- Coverage   14.73%   14.71%   -0.02%     
==========================================
  Files         200      200              
  Lines       93077    93027      -50     
==========================================
- Hits        13712    13692      -20     
+ Misses      78019    77989      -30     
  Partials     1346     1346              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@thaJeztah
thaJeztah requested review from corhere and vvoland August 1, 2026 17:35
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