Merge pull request #1751 from turetske/ticker_fix

Fixes an issue where a misplaced `defer ticker.Stop()` stopped the re…
PelicanPlatform · Nov 18, 2024 · cbcc668 · cbcc668
2 parents 334dc74 + 48e18b5
commit cbcc668
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/launcher_utils/advertise.go b/launcher_utils/advertise.go
@@ -68,9 +68,8 @@ func LaunchPeriodicAdvertise(ctx context.Context, egrp *errgroup.Group, servers
 	doAdvertise(ctx, servers)
 
 	ticker := time.NewTicker(1 * time.Minute)
-	defer ticker.Stop()
 	egrp.Go(func() error {
-
+		defer ticker.Stop()
 		for {
 			select {
 			case <-ticker.C: