From 8fabaa8f6e9bb58e39efdfc69929f0573505d6e9 Mon Sep 17 00:00:00 2001 From: Anthony Yeh Date: Sat, 25 Feb 2017 17:34:50 -0800 Subject: [PATCH] DaemonSet: Update Lister documentation for ControllerRef. The DaemonSet Listers still use selectors, because this is the behavior expected by callers. This clarifies the meaning of the returned list. Some callers may need to switch to using GetControllerOf() instead, but that is a separate, case-by-case issue. Kubernetes-commit: c288f52d2f3a05c305c9e5fb54ca01374b84ffa3 --- listers/extensions/v1beta1/daemonset_expansion.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/listers/extensions/v1beta1/daemonset_expansion.go b/listers/extensions/v1beta1/daemonset_expansion.go index dc17c105..019cdc40 100644 --- a/listers/extensions/v1beta1/daemonset_expansion.go +++ b/listers/extensions/v1beta1/daemonset_expansion.go @@ -35,8 +35,9 @@ type DaemonSetListerExpansion interface { // DaemonSetNamespaeLister. type DaemonSetNamespaceListerExpansion interface{} -// GetPodDaemonSets returns a list of daemon sets managing a pod. -// Returns an error if and only if no matching daemon sets are found. +// GetPodDaemonSets returns a list of DaemonSets that potentially match a pod. +// Only the one specified in the Pod's ControllerRef will actually manage it. +// Returns an error only if no matching DaemonSets are found. func (s *daemonSetLister) GetPodDaemonSets(pod *v1.Pod) ([]*v1beta1.DaemonSet, error) { var selector labels.Selector var daemonSet *v1beta1.DaemonSet