Skip to content

Commit

Permalink
minor bugfixes
Browse files Browse the repository at this point in the history
  • Loading branch information
dejanzele committed May 13, 2024
1 parent 01dc00b commit 02a31de
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ github.com/MarvinJWendt/testza v0.5.2/go.mod h1:xu53QFE5sCdjtMCKk8YMQ2MnymimEctc
github.com/atomicgo/cursor v0.0.1/go.mod h1:cBON2QmmrysudxNBFthvMtN32r3jxVRIvzkUiF/RuIk=
github.com/containerd/console v1.0.3 h1:lIr7SlA5PxZyMV30bDW0MGbiOPXwc63yRuCP0ARubLw=
github.com/containerd/console v1.0.3/go.mod h1:7LqA/THxQ86k76b8c/EMSiaJ3h1eZkMkXar0TQ1gf3U=
github.com/cpuguy83/go-md2man/v2 v2.0.3 h1:qMCsGGgs+MAzDFyp9LpAe1Lqy/fY/qCovCm0qnXZOBM=
github.com/cpuguy83/go-md2man/v2 v2.0.3/go.mod h1:tgQtvFlXSQOSOSIRvRPT7W67SCa46tRHOmNcaadrF8o=
github.com/cpuguy83/go-md2man/v2 v2.0.4 h1:wfIWP927BUkWJb2NmU/kNDYIBTh/ziUX91+lVfRxZq4=
github.com/cpuguy83/go-md2man/v2 v2.0.4/go.mod h1:tgQtvFlXSQOSOSIRvRPT7W67SCa46tRHOmNcaadrF8o=
Expand Down
4 changes: 2 additions & 2 deletions internal/k8s/manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ func (m *Manager) WaitForPodsToComplete(ctx context.Context, labelSelector strin
3*time.Hour,
true,
func(ctx context.Context) (done bool, err error) {
logger := logger.With("labelSelector", labelSelector)
logger = logger.With("labelSelector", labelSelector)
running := 0
completed := 0
logger.Info("checking if all pods are completed")
Expand All @@ -404,7 +404,7 @@ func (m *Manager) WaitForPodsToComplete(ctx context.Context, labelSelector strin
return false, err
}
logger.Info("listed pods", "count", len(pods.Items))
for _, pod := range pods.Items {
for _, pod := range pods.Items { //nolint
if pod.Status.Phase != corev1.PodSucceeded && pod.Status.Phase != corev1.PodFailed {
running++
} else {
Expand Down

0 comments on commit 02a31de

Please sign in to comment.