Merge pull request #1132 from Adirio/deprecation

k8s-ci-robot · web-flow · commit f74cbf4b4052 · 2019-10-29T12:44:53.000-07:00
Deprecated use removal from the book
diff --git a/docs/book/src/cronjob-tutorial/testdata/project/controllers/cronjob_controller.go b/docs/book/src/cronjob-tutorial/testdata/project/controllers/cronjob_controller.go
@@ -136,7 +136,7 @@ func (r *CronJobReconciler) Reconcile(req ctrl.Request) (ctrl.Result, error) {
 		set the namespace and field match (which is actually an index lookup that we set up below).
 	*/
 	var childJobs kbatch.JobList
-	if err := r.List(ctx, &childJobs, client.InNamespace(req.Namespace), client.MatchingField(jobOwnerKey, req.Name)); err != nil {
+	if err := r.List(ctx, &childJobs, client.InNamespace(req.Namespace), client.MatchingFields{jobOwnerKey: req.Name}); err != nil {
 		log.Error(err, "unable to list child Jobs")
 		return ctrl.Result{}, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ func (r *CronJobReconciler) Reconcile(req ctrl.Request) (ctrl.Result, error) {`
`136`	`136`	`set the namespace and field match (which is actually an index lookup that we set up below).`
`137`	`137`	`*/`
`138`	`138`	`var childJobs kbatch.JobList`
`139`		`- if err := r.List(ctx, &childJobs, client.InNamespace(req.Namespace), client.MatchingField(jobOwnerKey, req.Name)); err != nil {`
	`139`	`+ if err := r.List(ctx, &childJobs, client.InNamespace(req.Namespace), client.MatchingFields{jobOwnerKey: req.Name}); err != nil {`
`140`	`140`	`log.Error(err, "unable to list child Jobs")`
`141`	`141`	`return ctrl.Result{}, err`
`142`	`142`	`}`