Skip to content

Commit 9ce18ff

Browse files
committed
Simplify code in the pod deletion.
1 parent d9c0404 commit 9ce18ff

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

controllers/remove_pods.go

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -205,15 +205,14 @@ func includeInstance(r *FoundationDBClusterReconciler, context ctx.Context, clus
205205

206206
if len(addresses) > 0 {
207207
r.Recorder.Event(cluster, "Normal", "IncludingInstances", fmt.Sprintf("Including removed processes: %v", addresses))
208-
}
209208

210-
err = adminClient.IncludeInstances(addresses)
211-
if err != nil {
212-
return err
209+
err = adminClient.IncludeInstances(addresses)
210+
if err != nil {
211+
return err
212+
}
213213
}
214214

215-
needsSpecUpdate := cluster.Spec.PendingRemovals != nil
216-
if needsSpecUpdate {
215+
if cluster.Spec.PendingRemovals != nil {
217216
err := r.clearPendingRemovalsFromSpec(context, cluster)
218217
if err != nil {
219218
return err

0 commit comments

Comments
 (0)