Skip to content

Commit ca9d382

Browse files
committed
some tidying
1 parent 79d47c0 commit ca9d382

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

pkg/targetgroupbinding/resource_manager.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,6 @@ func (m *defaultResourceManager) reconcileWithIPTargetType(ctx context.Context,
183183
if err != nil {
184184
return "", "", false, ctrlerrors.NewErrorWithMetrics(controllerName, "list_targets_error", err, m.metricsCollector)
185185
}
186-
totalTargets := len(targets)
187186

188187
notDrainingTargets, _ := partitionTargetsByDrainingStatus(targets)
189188
matchedEndpointAndTargets, unmatchedEndpoints, unmatchedTargets := matchPodEndpointWithTargets(endpoints, notDrainingTargets)
@@ -244,7 +243,7 @@ func (m *defaultResourceManager) reconcileWithIPTargetType(ctx context.Context,
244243
return "", "", false, ctrlerrors.NewErrorWithMetrics(controllerName, "update_tracked_ip_targets_error", err, m.metricsCollector)
245244
}
246245

247-
eligibleTargetsCount := m.getMaxNewTargets(len(unmatchedEndpoints), totalTargets, tgbScopedLogger)
246+
eligibleTargetsCount := m.getMaxNewTargets(len(unmatchedEndpoints), len(targets), tgbScopedLogger)
248247
unmatchedEndpoints = unmatchedEndpoints[:eligibleTargetsCount]
249248

250249
if err := m.registerPodEndpoints(ctx, tgb, unmatchedEndpoints); err != nil {
@@ -314,7 +313,6 @@ func (m *defaultResourceManager) reconcileWithInstanceTargetType(ctx context.Con
314313
if err != nil {
315314
return "", "", false, ctrlerrors.NewErrorWithMetrics(controllerName, "list_targets_error", err, m.metricsCollector)
316315
}
317-
totalTargets := len(targets)
318316

319317
notDrainingTargets, _ := partitionTargetsByDrainingStatus(targets)
320318

@@ -349,7 +347,7 @@ func (m *defaultResourceManager) reconcileWithInstanceTargetType(ctx context.Con
349347
return "", "", false, ctrlerrors.NewErrorWithMetrics(controllerName, "update_tracked_instance_targets_error", err, m.metricsCollector)
350348
}
351349

352-
eligibleTargetsCount := m.getMaxNewTargets(len(unmatchedEndpoints), totalTargets, tgbScopedLogger)
350+
eligibleTargetsCount := m.getMaxNewTargets(len(unmatchedEndpoints), len(targets), tgbScopedLogger)
353351
unmatchedEndpoints = unmatchedEndpoints[:eligibleTargetsCount]
354352

355353
if err := m.registerNodePortEndpoints(ctx, tgb, unmatchedEndpoints); err != nil {

0 commit comments

Comments
 (0)