Merge pull request #20 from blakebarnett/bdb__limit_pod_list_by_names…

…pace

Limit Pod.List calls by namespace

Makefile

@@ -1,5 +1,5 @@
 name ?= goldpinger
-version ?= 1.0.3
+version ?= 1.1.0
 bin ?= goldpinger
 pkg ?= "github.com/bloomberg/goldpinger"
 tag = $(name):$(version)

pkg/goldpinger/k8s.go

@@ -16,14 +16,24 @@ package goldpinger
 
 import (
 	"log"
-
+	"io/ioutil"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
-func GetAllPods() map[string]string {
+func getNamespace() string {
+	b, err := ioutil.ReadFile("/var/run/secrets/kubernetes.io/serviceaccount/namespace")
+	if err != nil {
+		log.Println("Unable to determine namespace: ", err.Error())
+		return ""
+	}
+	namespace := string(b)
+	return namespace
+}
 
+func GetAllPods() map[string]string {
 	timer := GetLabeledKubernetesCallsTimer()
-	pods, err := GoldpingerConfig.KubernetesClient.CoreV1().Pods("").List(metav1.ListOptions{LabelSelector: "app=goldpinger"})
+	namespace := getNamespace()
+	pods, err := GoldpingerConfig.KubernetesClient.CoreV1().Pods(namespace).List(metav1.ListOptions{LabelSelector: "app=goldpinger"})
 	if err != nil {
 		log.Println("Error getting pods for selector: ", err.Error())
 		CountError("kubernetes_api")