-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update the documents with taint as pre-requisite #140
Conversation
Codecov Report
@@ Coverage Diff @@
## main #140 +/- ##
==========================================
+ Coverage 52.12% 52.37% +0.25%
==========================================
Files 6 6
Lines 1508 1516 +8
==========================================
+ Hits 786 794 +8
Misses 623 623
Partials 99 99
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM, thanks!
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.
LGTM
What type of PR is this?
/kind bug fix
What this PR does / why we need it:
This PR will bring in the following changes :
node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule
Which issue(s) this PR fixes:
Fixes #141
Test Report Added?:
Test Report:
1. When CPI is configured successfully
2. When ProviderID is not assigned