Merge pull request #705 from virtual-kubelet/fix-new-pod-controller
Make NewPodController function validate that provider is set
This commit is contained in:
@@ -138,6 +138,9 @@ func NewPodController(cfg PodControllerConfig) (*PodController, error) {
|
|||||||
if cfg.ServiceInformer == nil {
|
if cfg.ServiceInformer == nil {
|
||||||
return nil, errdefs.InvalidInput("missing service informer")
|
return nil, errdefs.InvalidInput("missing service informer")
|
||||||
}
|
}
|
||||||
|
if cfg.Provider == nil {
|
||||||
|
return nil, errdefs.InvalidInput("missing provider")
|
||||||
|
}
|
||||||
|
|
||||||
rm, err := manager.NewResourceManager(cfg.PodInformer.Lister(), cfg.SecretInformer.Lister(), cfg.ConfigMapInformer.Lister(), cfg.ServiceInformer.Lister())
|
rm, err := manager.NewResourceManager(cfg.PodInformer.Lister(), cfg.SecretInformer.Lister(), cfg.ConfigMapInformer.Lister(), cfg.ServiceInformer.Lister())
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|||||||
Reference in New Issue
Block a user