Merge pull request #2106 from adriangonz/remove-client-go

Remove executor's dependency on k8s.io/client-go
SeldonIO · Jul 13, 2020 · b8a9291 · b8a9291
2 parents 0562e15 + 3a93b7b
commit b8a9291
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 77 deletions.
diff --git a/executor/api/client/seldondeployment_client.go b/executor/api/client/seldondeployment_client.go
diff --git a/executor/go.mod b/executor/go.mod
@@ -26,7 +26,6 @@ require (
 	gotest.tools v2.2.0+incompatible
 	k8s.io/api v0.17.2
 	k8s.io/apimachinery v0.17.2
-	k8s.io/client-go v0.17.2
 	sigs.k8s.io/controller-runtime v0.5.0
 )
 

diff --git a/executor/main.go b/executor/main.go
@@ -237,14 +237,6 @@ func main() {
 		if err != nil {
 			logger.Error(err, "Failed to get predictor from Env")
 			panic(err)
-		} else if predictor == nil {
-			logger.Info("Trying to get predictor from API")
-			seldonDeploymentClient := seldonclient.NewSeldonDeploymentClient(configPath)
-			predictor, err = seldonDeploymentClient.GetPredictor(*sdepName, *namespace, *predictorName)
-			if err != nil {
-				logger.Error(err, "Failed to find predictor", "name", predictor)
-				panic(err)
-			}
 		}
 	}