Merge pull request kubernetes#80657 from rphillips/fixes/log_metrics_add_uid

kubelet: add UID to kubelet_container_log_filesystem_used_bytes metric

Author: k8s-ci-robot

pkg/kubelet/metrics/collectors/log_metrics.go

@@ -28,6 +28,7 @@ var (
 		"kubelet_container_log_filesystem_used_bytes",
 		"Bytes used by the container's logs on the filesystem.",
 		[]string{
+			"uid",
 			"namespace",
 			"pod",
 			"container",
@@ -67,6 +68,7 @@ func (c *logMetricsCollector) Collect(ch chan<- prometheus.Metric) {
 					descLogSize,
 					prometheus.GaugeValue,
 					float64(*c.Logs.UsedBytes),
+					ps.PodRef.UID,
 					ps.PodRef.Namespace,
 					ps.PodRef.Name,
 					c.Name,

pkg/kubelet/metrics/collectors/log_metrics_test.go

@@ -51,6 +51,7 @@ func TestMetricsCollected(t *testing.T) {
 					PodRef: statsapi.PodReference{
 						Namespace: "some-namespace",
 						Name:      "podName1",
+						UID:       "UID_some_id",
 					},
 					Containers: []statsapi.ContainerStats{
 						{
@@ -68,7 +69,7 @@ func TestMetricsCollected(t *testing.T) {
 	err := testutil.CollectAndCompare(collector, strings.NewReader(`
 		# HELP kubelet_container_log_filesystem_used_bytes Bytes used by the container's logs on the filesystem.
 		# TYPE kubelet_container_log_filesystem_used_bytes gauge
-		kubelet_container_log_filesystem_used_bytes{container="containerName1",namespace="some-namespace",pod="podName1"} 18
+		kubelet_container_log_filesystem_used_bytes{container="containerName1",namespace="some-namespace",pod="podName1",uid="UID_some_id"} 18
 `), "kubelet_container_log_filesystem_used_bytes")
 	if err != nil {
 		t.Fatal(err)