Merge pull request #4 from robscott/rs/better-auth

Adding More Auth Providers, Fixes for Certain Pod States

pkg/capacity/list.go

@@ -105,7 +105,9 @@ func buildClusterMetric(podList *corev1.PodList, pmList *v1beta1.PodMetricsList,
 	}
 
 	for _, pod := range podList.Items {
-		cm.addPodMetric(&pod)
+		if pod.Status.Phase != corev1.PodSucceeded && pod.Status.Phase != corev1.PodFailed {
+			cm.addPodMetric(&pod)
+		}
 	}
 
 	for _, node := range nodeList.Items {
@@ -122,9 +124,11 @@ func buildClusterMetric(podList *corev1.PodList, pmList *v1beta1.PodMetricsList,
 
 	for _, pod := range pmList.Items {
 		pm := cm.podMetrics[fmt.Sprintf("%s-%s", pod.GetNamespace(), pod.GetName())]
-		for _, container := range pod.Containers {
-			pm.cpu.utilization.Add(container.Usage["cpu"])
-			pm.memory.utilization.Add(container.Usage["memory"])
+		if pm != nil {
+			for _, container := range pod.Containers {
+				pm.cpu.utilization.Add(container.Usage["cpu"])
+				pm.memory.utilization.Add(container.Usage["memory"])
+			}
 		}
 	}
 

pkg/cmd/version.go

@@ -28,6 +28,6 @@ var versionCmd = &cobra.Command{
 	Use:   "version",
 	Short: "Print the version number of kube-capacity",
 	Run: func(cmd *cobra.Command, args []string) {
-		fmt.Println("kube-capacity version 0.1.2")
+		fmt.Println("kube-capacity version 0.1.3")
 	},
 }

pkg/kube/clientset.go

@@ -24,10 +24,8 @@ import (
 	"k8s.io/client-go/tools/clientcmd"
 	metrics "k8s.io/metrics/pkg/client/clientset/versioned"
 
-	// Required for GKE
-	_ "k8s.io/client-go/plugin/pkg/client/auth/gcp"
-	// Required for OIDC
-	_ "k8s.io/client-go/plugin/pkg/client/auth/oidc"
+	// Required for GKE, OIDC, and more
+	_ "k8s.io/client-go/plugin/pkg/client/auth"
 )
 
 // NewClientSet returns a new Kubernetes clientset