Unverified Commit e2c02f42 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #53970 from ScorpioCPH/add-more-comments

Automatic merge from submit-queue (batch tested with PRs 55283, 55461, 55288, 53970, 55487). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Add more comments for DevicePluginHandlerImpl struct **What this PR does / why we need it**: Add more comments **Special notes for your reviewer**: @jiayingz PTAL. **Release note**: ``` NONE ```
parents 74ec8d0f 1d4e1942
...@@ -61,12 +61,14 @@ type Handler interface { ...@@ -61,12 +61,14 @@ type Handler interface {
type HandlerImpl struct { type HandlerImpl struct {
// TODO: consider to change this to RWMutex. // TODO: consider to change this to RWMutex.
sync.Mutex sync.Mutex
// devicePluginManager is an implementation of deviceplugin.Manager interface.
devicePluginManager Manager devicePluginManager Manager
// activePods is a method for listing active pods on the node // activePods is a method for listing active pods on the node
// so the amount of pluginResources requested by existing pods // so the amount of pluginResources requested by existing pods
// could be counted when updating allocated devices // could be counted when updating allocated devices
activePods ActivePodsFunc activePods ActivePodsFunc
// devicePluginManagerMonitorCallback is used for testing only. // devicePluginManagerMonitorCallback is used for updating devices' states in one time call.
// e.g. a new device is advertised, two old devices are deleted and a running device fails.
devicePluginManagerMonitorCallback MonitorCallback devicePluginManagerMonitorCallback MonitorCallback
// allDevices contains all of registered resourceNames and their exported device IDs. // allDevices contains all of registered resourceNames and their exported device IDs.
allDevices map[string]sets.String allDevices map[string]sets.String
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment