Skip to content

Commit

Permalink
pr feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
npolshakova committed Jun 18, 2024
1 parent 6014304 commit b2fbb5c
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions projects/gateway2/status/status_syncer.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,9 +76,6 @@ func (f *statusSyncerFactory) QueueStatusForProxies(
f.resyncsPerProxy[getProxyNameNamespace(proxy)] = totalSyncCount

// keep track of proxies to check all proxies are handled in debugger
if f.resyncsPerIteration[totalSyncCount] == nil {
f.resyncsPerIteration[totalSyncCount] = make([]types.NamespacedName, 0)
}
f.resyncsPerIteration[totalSyncCount] = append(f.resyncsPerIteration[totalSyncCount], getProxyNameNamespace(proxy))
}
// the plugin registry that produced the proxies is the same for all proxies in a given sync
Expand Down Expand Up @@ -132,10 +129,6 @@ func (f *statusSyncerFactory) HandleProxyReports(ctx context.Context, proxiesWit
for syncCount, proxies := range proxiesToReport {
if plugins, ok := f.registryPerSync[syncCount]; ok {
newStatusSyncer(plugins).applyStatusPlugins(ctx, proxies)
} else {
// This can happen when a non-proxy resource is reconciled by the gloo proxy reconciler (upstreams, secrets, etc)
contextutils.LoggerFrom(ctx).Debugf("no registry found for proxy sync count %d", syncCount)
continue
}

// If there are no more proxies for the sync iteration, delete the sync count
Expand Down

0 comments on commit b2fbb5c

Please sign in to comment.