[CORE-119212] - fix lint and logging

Ha Van · Ha Van · commit 36c4b1950d74 · 2024-12-20T13:16:16.000-06:00
diff --git a/pkg/resources/kafka/poddisruptionbudget.go b/pkg/resources/kafka/poddisruptionbudget.go
@@ -60,6 +60,7 @@ func (r *Reconciler) podDisruptionBudgetControllers(log logr.Logger) (runtime.Ob
 	minAvailable, err := r.computeControllerMinAvailable()
 
 	if err != nil {
+		log.Error(err, "error occurred during computing minAvailable for controllers PDB")
 		return nil, err
 	}
 
@@ -132,7 +133,7 @@ func (r *Reconciler) computeMinAvailable(log logr.Logger) (intstr.IntOrString, e
 
 	controllerCount, err := r.getControllerCount()
 	if err != nil {
-		log.Error(err, "error occurred during parsing the disruption budget")
+		log.Error(err, "error occurred during get controller count")
 		return intstr.FromInt(-1), err
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,7 @@ func (r *Reconciler) podDisruptionBudgetControllers(log logr.Logger) (runtime.Ob`
`60`	`60`	`minAvailable, err := r.computeControllerMinAvailable()`
`61`	`61`
`62`	`62`	`if err != nil {`
	`63`	`+ log.Error(err, "error occurred during computing minAvailable for controllers PDB")`
`63`	`64`	`return nil, err`
`64`	`65`	`}`
`65`	`66`
`@@ -132,7 +133,7 @@ func (r *Reconciler) computeMinAvailable(log logr.Logger) (intstr.IntOrString, e`
`132`	`133`
`133`	`134`	`controllerCount, err := r.getControllerCount()`
`134`	`135`	`if err != nil {`
`135`		`- log.Error(err, "error occurred during parsing the disruption budget")`
	`136`	`+ log.Error(err, "error occurred during get controller count")`
`136`	`137`	`return intstr.FromInt(-1), err`
`137`	`138`	`}`
`138`	`139`