Adding ContainersPerPodCount metric.

This will let us know the composition of pods in clusters.
satnam6502 · Feb 27, 2015 · 556acc7 · 556acc7
1 parent 0141561
commit 556acc7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/pkg/kubelet/kubelet.go b/pkg/kubelet/kubelet.go
@@ -1306,6 +1306,11 @@ func (kl *Kubelet) SyncPods(pods []api.BoundPod, podSyncTypes map[types.UID]metr
 		kl.podWorkers.UpdatePod(pod, func() {
 			metrics.SyncPodLatency.WithLabelValues(podSyncTypes[pod.UID].String()).Observe(metrics.SinceInMicroseconds(start))
 		})
+
+		// Note the number of containers for new pods.
+		if val, ok := podSyncTypes[pod.UID]; ok && (val == metrics.SyncPodCreate) {
+			metrics.ContainersPerPodCount.Observe(float64(len(pod.Spec.Containers)))
+		}
 	}
 
 	// Stop the workers for no-longer existing pods.

diff --git a/pkg/kubelet/metrics/metrics.go b/pkg/kubelet/metrics/metrics.go
@@ -36,7 +36,13 @@ var (
 			Help:      "Image pull latency in microseconds.",
 		},
 	)
-	// TODO(vmarmol): Break down by number of containers in pod?
+	ContainersPerPodCount = prometheus.NewSummary(
+		prometheus.SummaryOpts{
+			Subsystem: kubeletSubsystem,
+			Name:      "containers_per_pod_count",
+			Help:      "The number of containers per pod.",
+		},
+	)
 	SyncPodLatency = prometheus.NewSummaryVec(
 		prometheus.SummaryOpts{
 			Subsystem: kubeletSubsystem,
@@ -52,7 +58,6 @@ var (
 			Help:      "Latency in microseconds to sync all pods.",
 		},
 	)
-	// TODO(vmarmol): Containers per pod
 	DockerOperationsLatency = prometheus.NewSummaryVec(
 		prometheus.SummaryOpts{
 			Subsystem: kubeletSubsystem,
@@ -73,6 +78,7 @@ func Register(containerCache dockertools.DockerCache) {
 		prometheus.MustRegister(SyncPodLatency)
 		prometheus.MustRegister(DockerOperationsLatency)
 		prometheus.MustRegister(SyncPodsLatency)
+		prometheus.MustRegister(ContainersPerPodCount)
 		prometheus.MustRegister(newPodAndContainerCollector(containerCache))
 	})
 }