-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Issue 8344: constrain data path expose #9064
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Lyndon-Li <[email protected]>
34145a9 to
bbcdb47
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9064 +/- ##
==========================================
- Coverage 60.31% 60.28% -0.04%
==========================================
Files 378 379 +1
Lines 42822 43066 +244
==========================================
+ Hits 25830 25964 +134
- Misses 15438 15548 +110
Partials 1554 1554 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
80a56d2 to
f815730
Compare
6bc1c71 to
0269c9e
Compare
Signed-off-by: Lyndon-Li <[email protected]>
0269c9e to
ecdb330
Compare
kaovilai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Configuration Validation: Add validation for PrepareQueueLength to ensure it's a
positive number
Documentation: The PR would benefit from:
- Documentation on how to configure PrepareQueueLength
The validation is here: Documentation will be in a separate PR after this one is merged. |
* issue 8344: constrain data path exposure. Signed-off-by: Lyndon-Li <[email protected]> Signed-off-by: Tiger Kaovilai <[email protected]>
* issue 8344: constrain data path exposure. Signed-off-by: Lyndon-Li <[email protected]>
Fix #8344, add a mechanism to soothe creation of data mover pods for DataUpload, DataDownload, PodVolumeBackup and PodVolumeRestore