github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-06 02:34:03 +00:00
Code Issues Projects Releases Wiki Activity

allow failed discovery on initial quota controller start

Browse Source
This commit is contained in:
David Eads 2018-08-15 08:19:24 -04:00
parent 6a12a280a5
commit 4c8e9de293
1 changed files with 22 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
pkg/controller/resourcequota/resource_quota_controller.go
View File

@ -161,11 +161,14 @@ func NewResourceQuotaController(options *ResourceQuotaControllerOptions) (*Resou
rq.quotaMonitor = qm rq.quotaMonitor = qm
// do initial quota monitor setup // do initial quota monitor setup. If we have a discovery failure here, it's ok. We'll discover more resources when a later sync happens.
resources, err := GetQuotableResources(options.DiscoveryFunc) resources, err := GetQuotableResources(options.DiscoveryFunc)
if err != nil { if discovery.IsGroupDiscoveryFailedError(err) {
utilruntime.HandleError(fmt.Errorf("initial discovery check failure, continuing and counting on future sync update: %v", err))
} else if err != nil {
return nil, err return nil, err
} }
if err = qm.SyncMonitors(resources); err != nil { if err = qm.SyncMonitors(resources); err != nil {
utilruntime.HandleError(fmt.Errorf("initial monitor sync has error: %v", err)) utilruntime.HandleError(fmt.Errorf("initial monitor sync has error: %v", err))
} }
@ -425,7 +428,16 @@ func (rq *ResourceQuotaController) Sync(discoveryFunc NamespacedResourcesFunc, p
newResources, err := GetQuotableResources(discoveryFunc) newResources, err := GetQuotableResources(discoveryFunc)
if err != nil { if err != nil {
utilruntime.HandleError(err) utilruntime.HandleError(err)
return
if discovery.IsGroupDiscoveryFailedError(err) && len(newResources) > 0 {
// In partial discovery cases, don't remove any existing informers, just add new ones
for k, v := range oldResources {
newResources[k] = v
}
} else {
// short circuit in non-discovery error cases or if discovery returned zero resources
return
}
} }
// Decide whether discovery has reported a change. // Decide whether discovery has reported a change.
@ -470,15 +482,18 @@ func (rq *ResourceQuotaController) resyncMonitors(resources map[schema.GroupVers
// GetQuotableResources returns all resources that the quota system should recognize. // GetQuotableResources returns all resources that the quota system should recognize.
// It requires a resource supports the following verbs: 'create','list','delete' // It requires a resource supports the following verbs: 'create','list','delete'
// This function may return both results and an error. If that happens, it means that the discovery calls were only
// partially successful. A decision about whether to proceed or not is left to the caller.
func GetQuotableResources(discoveryFunc NamespacedResourcesFunc) (map[schema.GroupVersionResource]struct{}, error) { func GetQuotableResources(discoveryFunc NamespacedResourcesFunc) (map[schema.GroupVersionResource]struct{}, error) {
possibleResources, err := discoveryFunc() possibleResources, discoveryErr := discoveryFunc()
if err != nil { if discoveryErr != nil && len(possibleResources) == 0 {
return nil, fmt.Errorf("failed to discover resources: %v", err) return nil, fmt.Errorf("failed to discover resources: %v", discoveryErr)
} }
quotableResources := discovery.FilteredBy(discovery.SupportsAllVerbs{Verbs: []string{"create", "list", "watch", "delete"}}, possibleResources) quotableResources := discovery.FilteredBy(discovery.SupportsAllVerbs{Verbs: []string{"create", "list", "watch", "delete"}}, possibleResources)
quotableGroupVersionResources, err := discovery.GroupVersionResources(quotableResources) quotableGroupVersionResources, err := discovery.GroupVersionResources(quotableResources)
if err != nil { if err != nil {
return nil, fmt.Errorf("Failed to parse resources: %v", err) return nil, fmt.Errorf("Failed to parse resources: %v", err)
} }
return quotableGroupVersionResources, nil // return the original discovery error (if any) in addition to the list
return quotableGroupVersionResources, discoveryErr
} }