fix(librarian): restrict update all to googleapis discovery (#3724)

Rust currently lists multiple sources (discovery, googleapis, showcase,
conformance, protobuf), but updating showcase, conformance, or protobuf
would require additional code changes in google-cloud-rust.
Update to restrict `librarian update --all` to only update googleapis
and discovery. And use it in `librarianops` command.

Fix #3691

Author: zhumin8

internal/librarian/update.go

@@ -18,8 +18,6 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"maps"
-	"slices"
 	"strings"
 
 	"github.com/googleapis/librarian/internal/config"
@@ -47,13 +45,19 @@ var (
 // updateCommand returns the `update` subcommand.
 func updateCommand() *cli.Command {
 	return &cli.Command{
-		Name:      "update",
-		Usage:     "update sources to the latest version",
+		Name:  "update",
+		Usage: "update sources to the latest version",
+		Description: `Supported sources are:
+  - conformance
+  - discovery
+  - googleapis
+  - protobuf
+  - showcase`,
 		UsageText: "librarian update [--all | source]",
 		Flags: []cli.Flag{
 			&cli.BoolFlag{
 				Name:  "all",
-				Usage: "update all sources",
+				Usage: "update discovery and googleapis sources",
 			},
 		},
 		Action: func(ctx context.Context, cmd *cli.Command) error {
@@ -104,7 +108,7 @@ func runUpdate(all bool, sourceName string) error {
 
 	var sourceNamesToProcess []string
 	if all {
-		sourceNamesToProcess = slices.Collect(maps.Keys(sourceRepos))
+		sourceNamesToProcess = []string{"discovery", "googleapis"}
 	} else {
 		sourceNamesToProcess = []string{sourceName}
 	}

internal/librarian/update_test.go

@@ -422,17 +422,17 @@ func TestUpdateCommand(t *testing.T) {
 						SHA256: "this-should-be-changed",
 					},
 					Conformance: &config.Source{
-						Commit: "this-should-be-changed",
-						SHA256: "this-should-be-changed",
+						Commit: "this-should-not-change",
+						SHA256: "this-should-not-change",
 					},
 					ProtobufSrc: &config.Source{
 						Branch: testBranch,
-						Commit: "this-should-be-changed",
-						SHA256: "this-should-be-changed",
+						Commit: "this-should-not-change",
+						SHA256: "this-should-not-change",
 					},
 					Showcase: &config.Source{
-						Commit: "this-should-be-changed",
-						SHA256: "this-should-be-changed",
+						Commit: "this-should-not-change",
+						SHA256: "this-should-not-change",
 					},
 				},
 			},
@@ -448,17 +448,17 @@ func TestUpdateCommand(t *testing.T) {
 						SHA256: discoveryTestSHA,
 					},
 					Conformance: &config.Source{
-						Commit: conformanceTestCommit,
-						SHA256: conformanceTestSHA,
+						Commit: "this-should-not-change",
+						SHA256: "this-should-not-change",
 					},
 					ProtobufSrc: &config.Source{
 						Branch: testBranch,
-						Commit: protobufTestCommit,
-						SHA256: protobufTestSHA,
+						Commit: "this-should-not-change",
+						SHA256: "this-should-not-change",
 					},
 					Showcase: &config.Source{
-						Commit: showcaseTestCommit,
-						SHA256: showcaseTestSHA,
+						Commit: "this-should-not-change",
+						SHA256: "this-should-not-change",
 					},
 				},
 			},

internal/librarianops/librarianops.go

@@ -76,12 +76,11 @@ For each repository, librarianops will:
   2. Create a branch: librarianops-generateall-YYYY-MM-DD
   3. Resolve librarian version from @main and update version field in librarian.yaml
   4. Run librarian tidy
-  5. Run librarian update discovery (google-cloud-rust only)
-  6. Run librarian update googleapis
-  7. Run librarian generate --all
-  8. Run cargo update --workspace (google-cloud-rust only)
-  9. Commit changes
-  10. Create a pull request`,
+  5. Run librarian update --all
+  6. Run librarian generate --all
+  7. Run cargo update --workspace (google-cloud-rust only)
+  8. Commit changes
+  9. Create a pull request`,
 		Flags: []cli.Flag{
 			&cli.BoolFlag{
 				Name:  "all",
@@ -169,13 +168,8 @@ func processRepo(ctx context.Context, repoName, repoDir string) (err error) {
 			return err
 		}
 	}
-	if repoName == repoRust {
-		if err := runLibrarianWithVersion(ctx, version, "update", "discovery"); err != nil {
-			return err
-		}
-	}
 	if repoName != repoFake {
-		if err := runLibrarianWithVersion(ctx, version, "update", "googleapis"); err != nil {
+		if err := runLibrarianWithVersion(ctx, version, "update", "--all"); err != nil {
 			return err
 		}
 	}