Fix the app and version names in the MultidimPodAutoscaler resources. (…

…#126)

These names have to be valid label names. Moreover, they have to match
the resources we're deleting.

Without these changes, the multidimpodautoscaler resources will end up
hanging.

internal/gke/gke.go

@@ -450,7 +450,7 @@ var autoJSONTmpl = template.Must(template.New("auto").Parse(`{
 // configured for the given Kubernetes ReplicaSet.
 func ensureReplicaSetAutoscaler(ctx context.Context, cluster *ClusterInfo, logger *slog.Logger, cfg *config.GKEConfig, replicaSet string) error {
 	dep := cfg.Deployment
-	name := name{dep.App.Name, replicaSet, dep.Id[:8]}.DNSLabel()
+	aName := name{dep.App.Name, replicaSet, dep.Id[:8]}.DNSLabel()
 	var b strings.Builder
 	if err := autoJSONTmpl.Execute(&b, struct {
 		Name             string
@@ -462,14 +462,14 @@ func ensureReplicaSetAutoscaler(ctx context.Context, cluster *ClusterInfo, logge
 		TargetKind       string
 		TargetName       string
 	}{
-		Name:             name,
+		Name:             aName,
 		Namespace:        namespaceName,
-		AppName:          dep.App.Name,
-		DeploymentID:     dep.Id,
+		AppName:          name{dep.App.Name}.DNSLabel(),
+		DeploymentID:     name{dep.Id}.DNSLabel(),
 		AppContainerName: appContainerName,
 		MinMemory:        memoryUnit.String(),
 		TargetKind:       "Deployment",
-		TargetName:       name,
+		TargetName:       aName,
 	}); err != nil {
 		return err
 	}

internal/gke/nanny.go

@@ -372,13 +372,14 @@ func RunManager(ctx context.Context, port int) error {
 		},
 		// deleteAppVersions
 		func(_ context.Context, app string, versions []*config.GKEConfig) error {
+			var errs []error
 			for _, version := range versions {
 				id := version.Deployment.Id
 				if err := kill(ctx, cluster, logger, app, id); err != nil {
-					return fmt.Errorf("kill %q: %w", version, err)
+					errs = append(errs, fmt.Errorf("kill %q: %w", version, err))
 				}
 			}
-			return nil
+			return errors.Join(errs...)
 		},
 	)