From 06eafce90142238d9e915ec7381d037cc44dc52d Mon Sep 17 00:00:00 2001 From: Matthias Diester Date: Mon, 11 Mar 2024 10:10:08 +0100 Subject: [PATCH] Apply suggestions from code review Fix variable usage in message. Co-authored-by: Sascha Schwarze --- pkg/validate/strategies.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/validate/strategies.go b/pkg/validate/strategies.go index 5bbc99541d..b02086838d 100644 --- a/pkg/validate/strategies.go +++ b/pkg/validate/strategies.go @@ -69,7 +69,7 @@ func (s Strategy) validateBuildStrategy(ctx context.Context, strategyName string if apierrors.IsNotFound(err) { s.Build.Status.Reason = build.BuildReasonPtr(build.BuildStrategyNotFound) - s.Build.Status.Message = pointer.String(fmt.Sprintf("buildStrategy %s does not exist in namespace %s", s.Build.Spec.Strategy.Name, s.Build.Namespace)) + s.Build.Status.Message = pointer.String(fmt.Sprintf("buildStrategy %s does not exist in namespace %s", strategyName, s.Build.Namespace)) return nil } @@ -87,7 +87,7 @@ func (s Strategy) validateClusterBuildStrategy(ctx context.Context, strategyName if apierrors.IsNotFound(err) { s.Build.Status.Reason = build.BuildReasonPtr(build.ClusterBuildStrategyNotFound) - s.Build.Status.Message = pointer.String(fmt.Sprintf("clusterBuildStrategy %s does not exist", s.Build.Spec.Strategy.Name)) + s.Build.Status.Message = pointer.String(fmt.Sprintf("clusterBuildStrategy %s does not exist", strategyName)) return nil }