entrypoint/src/cmd/entrypoint/garage.go

@@ -36,12 +36,15 @@ var subCmdGarageMC = subCmd{
 
 		s3APIAddr := hostBootstrap.ChooseGaragePeer().S3APIAddr()
 
-		if *keyID == "" {
-			*keyID = hostBootstrap.Garage.GlobalBucketS3APICredentials.ID
-		}
+		if *keyID == "" || *keySecret == "" {
 
-		if *keySecret == "" {
-			*keySecret = hostBootstrap.Garage.GlobalBucketS3APICredentials.Secret
+			if *keyID == "" {
+				*keyID = hostBootstrap.Garage.GlobalBucketS3APICredentials.ID
+			}
+
+			if *keySecret == "" {
+				*keyID = hostBootstrap.Garage.GlobalBucketS3APICredentials.Secret
+			}
 		}
 
 		args := flags.Args()
@@ -53,15 +56,13 @@ var subCmdGarageMC = subCmd{
 		args = append([]string{"mc"}, args...)
 
 		var (
-			mcHostVar = fmt.Sprintf(
-				"MC_HOST_garage=http://%s:%s@%s",
-				*keyID, *keySecret, s3APIAddr,
-			)
-
 			binPath = filepath.Join(envAppDirPath, "bin/mc")
 			cliEnv  = append(
 				os.Environ(),
-				mcHostVar,
+				fmt.Sprintf(
+					"MC_HOST_garage=http://%s:%s@%s",
+					*keyID, *keySecret, s3APIAddr,
+				),
 
 				// The garage docs say this is necessary, though nothing bad
 				// seems to happen if we leave it out *shrug*

entrypoint/src/cmd/entrypoint/garage_util.go

@@ -265,14 +265,9 @@ func garageApplyLayout(
 
 			id := bootstrapGarageHostForAlloc(thisHost, alloc).ID
 
-			zone := hostName
-			if alloc.Zone != "" {
-				zone = alloc.Zone
-			}
-
 			clusterLayout[id] = peerLayout{
 				Capacity: alloc.Capacity / 100,
-				Zone:     zone,
+				Zone:     hostName,
 				Tags:     []string{},
 			}
 		}

entrypoint/src/daemon/config.go

@@ -36,10 +36,6 @@ type ConfigStorageAllocation struct {
 	S3APIPort int    `yaml:"s3_api_port"`
 	RPCPort   int    `yaml:"rpc_port"`
 	AdminPort int    `yaml:"admin_port"`
-
-	// Zone is a secret option which makes it easier to test garage bugs, but
-	// which we don't want users to otherwise know about.
-	Zone string `yaml:"zone"`
 }
 
 // Config describes the structure of the daemon config file.