Fix the issue of incorrect container resource specs (#30)

Co-authored-by: able8 <able.lv@airwallex.com>

Author: able8

CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.2.1] - 2023-04-27
+### Fixed
+- Container resource specs showing wrong values [#26](https://github.com/airwallex/k8s-pod-restart-info-collector/issues/26)
+
+### Improved
+- Add backticks to format slack message nicely [#25](https://github.com/airwallex/k8s-pod-restart-info-collector/issues/25)
 
 ## [1.2.0] - 2023-01-03
 ### Added

build.sh

@@ -1,5 +1,5 @@
 #!/bin/bash
-TAG="v1.2.0"
+TAG="v1.2.1"
 docker buildx build --platform linux/amd64 -t devopsairwallex/k8s-pod-restart-info-collector:${TAG} .
 docker push devopsairwallex/k8s-pod-restart-info-collector:${TAG}
 

controller.go

@@ -213,7 +213,7 @@ func (c *Controller) handlePod(pod *v1.Pod) error {
 	}
 
 	// check and collect restarted container info
-	for i, status := range pod.Status.ContainerStatuses {
+	for _, status := range pod.Status.ContainerStatuses {
 		if status.RestartCount == 0 {
 			continue
 		}
@@ -232,7 +232,13 @@ func (c *Controller) handlePod(pod *v1.Pod) error {
 
 		restartReason := printContainerLastStateReason(status)
 
-		containerSpec := pod.Spec.Containers[i]
+		var containerSpec v1.Container
+		for _, container := range pod.Spec.Containers {
+			if status.Name == container.Name {
+				containerSpec = container
+				break
+			}
+		}
 		containerResource, err := getContainerResource(containerSpec)
 		if err != nil {
 			return err