Skip to content

Commit

Permalink
universe: rename FederationEnvoy.syncUniverseState to syncServerState
Browse files Browse the repository at this point in the history
Rename reasoning: this method syncs with a server and does not sync with
a universe. It will be used to sync multiple universe states with a
single server from within the federation.
  • Loading branch information
ffranr committed Oct 11, 2023
1 parent f556a87 commit 13b097f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions universe/auto_syncer.go
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ func (f *FederationEnvoy) reportErr(err error) {
// syncUniverseState attempts to sync Universe state with the target server.
// If the sync is successful (even if no diff is generated), then a new sync
// event will be logged.
func (f *FederationEnvoy) syncUniverseState(ctx context.Context,
func (f *FederationEnvoy) syncServerState(ctx context.Context,
addr ServerAddr) error {

log.Infof("Syncing Universe state with server=%v", spew.Sdump(addr))
Expand Down Expand Up @@ -468,7 +468,7 @@ func (f *FederationEnvoy) SyncServers(serverAddrs []ServerAddr) error {
ctx, cancel := f.WithCtxQuitNoTimeout()
defer cancel()

err := fn.ParSlice(ctx, serverAddrs, f.syncUniverseState)
err := fn.ParSlice(ctx, serverAddrs, f.syncServerState)
if err != nil {
log.Warnf("unable to sync with server: %w", err)
}
Expand Down

0 comments on commit 13b097f

Please sign in to comment.