From ce60fb81d4e7c7214bd96708a212f3a7115cf819 Mon Sep 17 00:00:00 2001
From: Sargun Dhillon <sargun@sargun.me>
Date: Sun, 21 Jul 2019 16:19:00 -0700
Subject: [PATCH] Make NewPodController function validate that provider is set

In NewPodController we validate that the rest of the config is
set to non-nil values. The provider must be non-nil as well.
---
 node/podcontroller.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/node/podcontroller.go b/node/podcontroller.go
index dda3b57a6..fdbb89734 100644
--- a/node/podcontroller.go
+++ b/node/podcontroller.go
@@ -138,6 +138,9 @@ func NewPodController(cfg PodControllerConfig) (*PodController, error) {
 	if cfg.ServiceInformer == nil {
 		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())
 	if err != nil {