Skip to content

Commit eba3ff3

Browse files
committed
Fix linting errors
1 parent 8359d87 commit eba3ff3

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

internal/pkg/handler/upgrade.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,11 @@ func PerformAction(clients kube.Clients, config util.Config, upgradeFuncs callba
291291
if !ok {
292292
logrus.Warnf("Annotation '%s' only applicable for deployments", options.PauseDeploymentAnnotation)
293293
} else {
294-
PauseDeployment(deployment, clients, itemName, itemNamespace, pauseInterval)
294+
err = PauseDeployment(deployment, clients, itemName, itemNamespace, pauseInterval)
295+
if err != nil {
296+
logrus.Errorf("Failed to pause deployment '%s' in namespace '%s': %v", itemName, itemNamespace, err)
297+
return err
298+
}
295299
}
296300
}
297301

internal/pkg/handler/upgrade_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3633,7 +3633,7 @@ func testPausingDeployment(t *testing.T, reloadStrategy string, testName string,
36333633
items := deploymentFuncs.ItemsFunc(clients, config.Namespace)
36343634
deploymentPaused, err := isDeploymentPaused(items, testName)
36353635
if err != nil {
3636-
t.Errorf(err.Error())
3636+
t.Errorf("%s", err.Error())
36373637
}
36383638
if !deploymentPaused {
36393639
t.Errorf("Deployment has not been paused")

0 commit comments

Comments
 (0)