-
Notifications
You must be signed in to change notification settings - Fork 81
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
Integrating sr iov dp #50
Conversation
Introduce VfDeviceID in EP Spec Import Multus checkpoint code (still not used)
Change log to error in getRegisteredDevices function Remove outdated TODO entries
…ntel SR IOV CNI plugin
…Multus; update glide.lock
polish some error messages
…ConfigMap solution; Refactor SetupNetworking() function accordingly
1161d7a
to
cd9ab94
Compare
Okay, logic is now super-duper. let's also find out why netwatcher ignores your new param. you know that you use device_pool as the name of your attribute in your code, right? weren't you using "resource_pool" in your test examples? |
Thanks for comments. Code related stuffs have been corrected. README.md update is going to be happened soon. CRD issue: You're right, epic fail :/ |
cedfa56
to
4491047
Compare
So, all-in-all, thanks! I think it was a good journey :) |
I agree :) Thank you for driving me to reach this point. |
What type of PR is this?
feature
What does this PR give to us:
DANM Integration with Intel SR-IOV Network Device Plugin and latest Intel SR-IOV CNI
Which issue(s) this PR fixes (in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: