Skip to content

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

Merged
thaJeztah merged 2 commits into
moby:masterfrom
thaJeztah:csi_unexport
Aug 26, 2026
Merged

agent/csi/plugin, manager/csi: un-export nodePlugin.Client#3278
thaJeztah merged 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

@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

codecov-commenter commented Aug 1, 2026

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 (5a92899) to head (931b472).
⚠️ Report is 18 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3278      +/-   ##
==========================================
- Coverage   14.76%   14.71%   -0.06%     
==========================================
  Files         200      200              
  Lines       93027    93027              
==========================================
- Hits        13736    13685      -51     
- Misses      77961    78003      +42     
- Partials     1330     1339       +9     
🚀 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
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 merged commit f441438 into moby:master Aug 26, 2026
14 of 16 checks passed
@thaJeztah
thaJeztah deleted the csi_unexport branch August 26, 2026 15:25
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.

3 participants