Skip to content

Commit

Permalink
Merge pull request #552 from mengqiy/fixsimplecontroller
Browse files Browse the repository at this point in the history
🐛 fix SimpleController in builder test
  • Loading branch information
k8s-ci-robot committed Aug 6, 2019
2 parents 054ee41 + 11818d8 commit 084b73e
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion pkg/builder/controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,12 @@ var _ = Describe("application", func() {
}
return nil, fmt.Errorf("max concurrent reconcilers expected %d but found %d", maxConcurrentReconciles, options.MaxConcurrentReconciles)
}
instance, err := SimpleController().

By("creating a controller manager")
m, err := manager.New(cfg, manager.Options{})
Expect(err).NotTo(HaveOccurred())

instance, err := ControllerManagedBy(m).
For(&appsv1.ReplicaSet{}).
Owns(&appsv1.ReplicaSet{}).
WithOptions(controller.Options{MaxConcurrentReconciles: maxConcurrentReconciles}).
Expand Down

0 comments on commit 084b73e

Please sign in to comment.