Skip to content

Commit

Permalink
golint
Browse files Browse the repository at this point in the history
Signed-off-by: Feny Mehta <[email protected]>
  • Loading branch information
fbm3307 committed Sep 13, 2024
1 parent aeef8de commit f2c29ee
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions pkg/cmd/adm/restart.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ func deletePods(ctx *clicontext.CommandContext, cl runtimeclient.Client, deploym

//delete pods
for _, pod := range pods.Items {
pod := pod // TODO We won't need it after upgrading to go 1.22: https://go.dev/blog/loopvar-preview
if err := cl.Delete(ctx, &pod); err != nil {
return err
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/cmd/adm/restart_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ func TestRestartHostOperator(t *testing.T) {
t.Run("host deployment is present and restart successful", func(t *testing.T) {
// given
deployment := newDeployment(namespacedName, 1)
deployment.Labels = map[string]string{"olm.owner.kind": "ClusterServiceVersion"}
deployment.Labels = map[string]string{"provider": "codeready-toolchain"}
newClient, fakeClient := NewFakeClients(t, deployment)
numberOfUpdateCalls := 0
fakeClient.MockUpdate = requireDeploymentBeingUpdated(t, fakeClient, namespacedName, 1, &numberOfUpdateCalls)
Expand Down

0 comments on commit f2c29ee

Please sign in to comment.