Job: Update Lister documentation for ControllerRef.

The Job 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: ad026026e0252469bab05f9adba4bb1b20a8f6a9
This commit is contained in:
Anthony Yeh 2017-02-26 18:28:26 -08:00 committed by Kubernetes Publisher
parent 69f0a81e21
commit 4c84996bc2

View File

@ -28,13 +28,17 @@ import (
// JobListerExpansion allows custom methods to be added to // JobListerExpansion allows custom methods to be added to
// JobLister. // JobLister.
type JobListerExpansion interface { type JobListerExpansion interface {
// GetPodJobs returns a list of jobs managing a pod. An error is returned only // GetPodJobs returns a list of Jobs that potentially
// if no matching jobs are found. // match a Pod. Only the one specified in the Pod's ControllerRef
// will actually manage it.
// Returns an error only if no matching Jobs are found.
GetPodJobs(pod *v1.Pod) (jobs []batch.Job, err error) GetPodJobs(pod *v1.Pod) (jobs []batch.Job, err error)
} }
// GetPodJobs returns a list of jobs managing a pod. An error is returned only // GetPodJobs returns a list of Jobs that potentially
// if no matching jobs are found. // match a Pod. Only the one specified in the Pod's ControllerRef
// will actually manage it.
// Returns an error only if no matching Jobs are found.
func (l *jobLister) GetPodJobs(pod *v1.Pod) (jobs []batch.Job, err error) { func (l *jobLister) GetPodJobs(pod *v1.Pod) (jobs []batch.Job, err error) {
if len(pod.Labels) == 0 { if len(pod.Labels) == 0 {
err = fmt.Errorf("no jobs found for pod %v because it has no labels", pod.Name) err = fmt.Errorf("no jobs found for pod %v because it has no labels", pod.Name)