update: improve informational printouts

Add and update informational printouts in 'git-bundle-server update' to
inform users more accurately of which step is in progress, as well as
clearly indicate when the update stops early due to a lack of updates from
the remote.

Also add a printout to 'update-all' indicating which route is being updated
at a given time as it loops through all routes.

Signed-off-by: Victoria Dye <[email protected]>

Author: vdye

cmd/git-bundle-server/update-all.go

@@ -56,6 +56,7 @@ func (u *updateAllCmd) Run(ctx context.Context, args []string) error {
 
 	for route := range repos {
 		subargs[1] = route
+		fmt.Printf("*** Updating %s ***\n", route)
 		exitCode, err := commandExecutor.RunStdout(ctx, exe, subargs...)
 		if err != nil {
 			return u.logger.Error(ctx, err)

cmd/git-bundle-server/update.go

@@ -52,30 +52,32 @@ func (u *updateCmd) Run(ctx context.Context, args []string) error {
 		return u.logger.Errorf(ctx, "failed to load bundle list: %w", err)
 	}
 
-	fmt.Printf("Creating new incremental bundle\n")
+	fmt.Printf("Checking for updates to %s\n", repo.Route)
 	bundle, err := bundleProvider.CreateIncrementalBundle(ctx, repo, list)
 	if err != nil {
 		return u.logger.Error(ctx, err)
 	}
 
 	// Nothing new!
 	if bundle == nil {
+		fmt.Printf("%s is up-to-date, no new bundles generated\n", repo.Route)
 		return nil
 	}
 
 	list.Bundles[bundle.CreationToken] = *bundle
 
-	fmt.Printf("Collapsing bundle list\n")
+	fmt.Println("Updating bundle list")
 	err = bundleProvider.CollapseList(ctx, repo, list)
 	if err != nil {
 		return u.logger.Error(ctx, err)
 	}
 
-	fmt.Printf("Writing updated bundle list\n")
+	fmt.Println("Writing updated bundle list")
 	listErr := bundleProvider.WriteBundleList(ctx, list, repo)
 	if listErr != nil {
 		return u.logger.Errorf(ctx, "failed to write bundle list: %w", listErr)
 	}
 
+	fmt.Println("Update complete")
 	return nil
 }