Register imagePullThroughput and count with MiB

Signed-off-by: Shukui Yang <yangshukui@bytedance.com>
(cherry picked from commit db22327)

pkg/cri/sbserver/image_pull.go

@@ -213,8 +213,9 @@ func (c *criService) PullImage(ctx context.Context, r *runtime.PullImageRequest)
 		}
 	}
 
+	const mbToByte = 1024 * 1024
 	size, _ := image.Size(ctx)
-	imagePullingSpeed := float64(size) / time.Since(startTime).Seconds()
+	imagePullingSpeed := float64(size) / mbToByte / time.Since(startTime).Seconds()
 	imagePullThroughput.Observe(imagePullingSpeed)
 
 	log.G(ctx).Infof("Pulled image %q with image id %q, repo tag %q, repo digest %q, size %q in %s", imageRef, imageID,

pkg/cri/sbserver/metrics.go

@@ -42,13 +42,18 @@ var (
 
 	imagePulls           metrics.LabeledCounter
 	inProgressImagePulls metrics.Gauge
-	//  pull duration / (image size / 1MBi)
+	// image size in MB / image pull duration in seconds
 	imagePullThroughput prom.Histogram
 )
 
 func init() {
+	const (
+		namespace = "containerd"
+		subsystem = "cri_sandboxed"
+	)
+
 	// these CRI metrics record latencies for successful operations around a sandbox and container's lifecycle.
-	ns := metrics.NewNamespace("containerd", "cri_sandboxed", nil)
+	ns := metrics.NewNamespace(namespace, subsystem, nil)
 
 	sandboxListTimer = ns.NewTimer("sandbox_list", "time to list sandboxes")
 	sandboxCreateNetworkTimer = ns.NewTimer("sandbox_create_network", "time to create the network for a sandbox")
@@ -72,12 +77,14 @@ func init() {
 	inProgressImagePulls = ns.NewGauge("in_progress_image_pulls", "in progress pulls", metrics.Total)
 	imagePullThroughput = prom.NewHistogram(
 		prom.HistogramOpts{
-			Name:    "image_pulling_throughput",
-			Help:    "image pull throughput",
-			Buckets: prom.DefBuckets,
+			Namespace: namespace,
+			Subsystem: subsystem,
+			Name:      "image_pulling_throughput",
+			Help:      "image pull throughput",
+			Buckets:   prom.DefBuckets,
 		},
 	)
-
+	ns.Add(imagePullThroughput)
 	metrics.Register(ns)
 }
 

pkg/cri/server/image_pull.go

@@ -215,8 +215,9 @@ func (c *criService) PullImage(ctx context.Context, r *runtime.PullImageRequest)
 		}
 	}
 
+	const mbToByte = 1024 * 1024
 	size, _ := image.Size(ctx)
-	imagePullingSpeed := float64(size) / time.Since(startTime).Seconds()
+	imagePullingSpeed := float64(size) / mbToByte / time.Since(startTime).Seconds()
 	imagePullThroughput.Observe(imagePullingSpeed)
 
 	log.G(ctx).Infof("Pulled image %q with image id %q, repo tag %q, repo digest %q, size %q in %s", imageRef, imageID,

pkg/cri/server/metrics.go

@@ -42,13 +42,18 @@ var (
 
 	imagePulls           metrics.LabeledCounter
 	inProgressImagePulls metrics.Gauge
-	//  pull duration / (image size / 1MBi)
+	// image size in MB / image pull duration in seconds
 	imagePullThroughput prom.Histogram
 )
 
 func init() {
+	const (
+		namespace = "containerd"
+		subsystem = "cri"
+	)
+
 	// these CRI metrics record latencies for successful operations around a sandbox and container's lifecycle.
-	ns := metrics.NewNamespace("containerd", "cri", nil)
+	ns := metrics.NewNamespace(namespace, subsystem, nil)
 
 	sandboxListTimer = ns.NewTimer("sandbox_list", "time to list sandboxes")
 	sandboxCreateNetworkTimer = ns.NewTimer("sandbox_create_network", "time to create the network for a sandbox")
@@ -72,12 +77,15 @@ func init() {
 	inProgressImagePulls = ns.NewGauge("in_progress_image_pulls", "in progress pulls", metrics.Total)
 	imagePullThroughput = prom.NewHistogram(
 		prom.HistogramOpts{
-			Name:    "image_pulling_throughput",
-			Help:    "image pull throughput",
-			Buckets: prom.DefBuckets,
+			Namespace: namespace,
+			Subsystem: subsystem,
+			Name:      "image_pulling_throughput",
+			Help:      "image pull throughput",
+			Buckets:   prom.DefBuckets,
 		},
 	)
 
+	ns.Add(imagePullThroughput)
 	metrics.Register(ns)
 }