Returns error when platform option is specified when running a bootst…

…rap build
hashicorp · Nov 25, 2024 · a835519 · a835519
1 parent 0b7f053
commit a835519
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/builder/docker/config.go b/builder/docker/config.go
@@ -141,8 +141,7 @@ type Config struct {
 	WindowsContainer bool `mapstructure:"windows_container" required:"false"`
 	// Set platform if server is multi-platform capable
 	//
-	// If using `build`, this field will be ignored, as the `platform` option for
-	// this operation will instead have precedence.
+	// This cannot be used at the same time as `build`; instead, use `build.platform`
 	Platform string `mapstructure:"platform" required:"false"`
 
 	// This is used to login to dockerhub to pull a private base container. For
@@ -224,7 +223,7 @@ func (c *Config) Prepare(raws ...interface{}) ([]string, error) {
 		}
 
 		if c.Platform != "" {
-			warnings = append(warnings, "when running a bootstrap build, the `platform` option is ignored and is replaced by `build.platform`")
+			errs = packersdk.MultiErrorAppend(errs, errors.New("when running a bootstrap build, the `platform` option cannot be specified (use `build.platform` instead)"))
 		}
 		c.Platform = c.BuildConfig.Platform
 	} else {

diff --git a/docs-partials/builder/docker/Config-not-required.mdx b/docs-partials/builder/docker/Config-not-required.mdx
@@ -100,8 +100,7 @@
 
 - `platform` (string) - Set platform if server is multi-platform capable
 
-  If using `build`, this field will be ignored, as the `platform` option for
-  this operation will instead have precedence.
+  This cannot be used at the same time as `build`; instead, use `build.platform`
 
 - `login` (bool) - This is used to login to dockerhub to pull a private base container. For
   pushing to dockerhub, see the docker post-processors