add rng device: to be squashed

hashicorp · May 26, 2023 · b4827ee · b4827ee
1 parent 6807697
commit b4827ee
Show file tree

Hide file tree

Showing 10 changed files with 100 additions and 1 deletion.
diff --git a/builder/proxmox/clone/config.hcl2spec.go b/builder/proxmox/clone/config.hcl2spec.go
diff --git a/builder/proxmox/common/config.go b/builder/proxmox/common/config.go
@@ -2,7 +2,7 @@
 // SPDX-License-Identifier: MPL-2.0
 
 //go:generate packer-sdc struct-markdown
-//go:generate packer-sdc mapstructure-to-hcl2 -type Config,NICConfig,diskConfig,vgaConfig,additionalISOsConfig,efiConfig
+//go:generate packer-sdc mapstructure-to-hcl2 -type Config,NICConfig,diskConfig,rng0Config,vgaConfig,additionalISOsConfig,efiConfig
 
 package proxmox
 
@@ -59,6 +59,7 @@ type Config struct {
 	EFIConfig      efiConfig      `mapstructure:"efi_config"`
 	EFIDisk        string         `mapstructure:"efidisk"`
 	Machine        string         `mapstructure:"machine"`
+	Rng0           rng0Config     `mapstructure:"rng0"`
 	VGA            vgaConfig      `mapstructure:"vga"`
 	NICs           []NICConfig    `mapstructure:"network_adapters"`
 	Disks          []diskConfig   `mapstructure:"disks"`
@@ -114,6 +115,13 @@ type efiConfig struct {
 	PreEnrolledKeys bool   `mapstructure:"pre_enrolled_keys"`
 	EFIType         string `mapstructure:"efi_type"`
 }
+
+type rng0Config struct {
+	Source   string `mapstructure:"source"`
+	MaxBytes int    `mapstructure:"max_bytes"`
+	Period   int    `mapstructure:"period"`
+}
+
 type vgaConfig struct {
 	Type   string `mapstructure:"type"`
 	Memory int    `mapstructure:"memory"`
@@ -391,6 +399,18 @@ func (c *Config) Prepare(upper interface{}, raws ...interface{}) ([]string, []st
 			errs = packersdk.MultiErrorAppend(errs, errors.New("efi_storage_pool not set for efi_config"))
 		}
 	}
+	if c.Rng0.Source != "" {
+		if !(c.Rng0.Source == "/dev/urandom" || c.Rng0.Source == "/dev/random" || c.Rng0.Source == "/dev/hwrng") {
+		        errs = packersdk.MultiErrorAppend(errs, errors.New("source must be one of \"/dev/urandom\", \"/dev/random\", \"/dev/hwrng\""))
+		}
+		// FIXME might be not declared by the user then not send to proxmox PVE API
+		if c.Rng0.MaxBytes < 0 {
+		        errs = packersdk.MultiErrorAppend(errs, errors.New("max_bytes must be greather or equal than 0"))
+		}
+		if !(c.Rng0.Period > 0) {
+		        errs = packersdk.MultiErrorAppend(errs, errors.New("period must be greather than 0"))
+		}
+	}
 
 	if errs != nil && len(errs.Errors) > 0 {
 		return nil, warnings, errs

diff --git a/builder/proxmox/common/config.hcl2spec.go b/builder/proxmox/common/config.hcl2spec.go
diff --git a/builder/proxmox/common/config_test.go b/builder/proxmox/common/config_test.go
@@ -246,6 +246,8 @@ func TestAdditionalISOs(t *testing.T) {
 
 }
 
+// FIXME Tests for Rng0
+
 func TestSerials(t *testing.T) {
 	serialsTest := []struct {
 		name          string

diff --git a/builder/proxmox/common/step_start_vm.go b/builder/proxmox/common/step_start_vm.go
@@ -123,6 +123,7 @@ func (s *stepStartVM) Run(ctx context.Context, state multistep.StateBag) multist
 		Bios:         c.BIOS,
 		EFIDisk:      generateProxmoxEfi(c.EFIConfig),
 		Machine:      c.Machine,
+		RNGDrive:     generateProxmoxRng0(c.Rng0),
 		QemuVga:      generateProxmoxVga(c.VGA),
 		QemuNetworks: generateProxmoxNetworkAdapters(c.NICs),
 		QemuDisks:    generateProxmoxDisks(c.Disks),
@@ -295,6 +296,19 @@ func generateProxmoxSerials(serials []string) proxmox.QemuDevices {
 	return devs
 }
 
+func generateProxmoxRng0(rng0 rng0Config) proxmox.QemuDevice {
+	dev := make(proxmox.QemuDevice)
+	setDeviceParamIfDefined(dev, "source", rng0.Source)
+
+	if rng0.MaxBytes > 0 {
+		dev["max_bytes"] = rng0.MaxBytes
+	}
+	if rng0.Period > 0 {
+		dev["period"] = rng0.Period
+	}
+	return dev
+}
+
 func generateProxmoxVga(vga vgaConfig) proxmox.QemuDevice {
 	dev := make(proxmox.QemuDevice)
 	setDeviceParamIfDefined(dev, "type", vga.Type)

diff --git a/builder/proxmox/iso/config.hcl2spec.go b/builder/proxmox/iso/config.hcl2spec.go
diff --git a/docs-partials/builder/proxmox/common/Config-not-required.mdx b/docs-partials/builder/proxmox/common/Config-not-required.mdx
@@ -44,6 +44,8 @@
 
 - `machine` (string) - Machine
 
+- `rng0` (rng0Config) - Rng 0
+
 - `vga` (vgaConfig) - VGA
 
 - `network_adapters` ([]NICConfig) - NI Cs

diff --git a/docs-partials/builder/proxmox/common/rng0Config-not-required.mdx b/docs-partials/builder/proxmox/common/rng0Config-not-required.mdx
@@ -0,0 +1,9 @@
+<!-- Code generated from the comments of the rng0Config struct in builder/proxmox/common/config.go; DO NOT EDIT MANUALLY -->
+
+- `source` (string) - Source
+
+- `max_bytes` (int) - Max Bytes
+
+- `period` (int) - Period
+
+<!-- End of code generated from the comments of the rng0Config struct in builder/proxmox/common/config.go; -->
diff --git a/docs/builders/clone.mdx b/docs/builders/clone.mdx
@@ -354,6 +354,23 @@ or responding to pattern `/dev/.+`. Example:
 
 - `machine` - (string) - Set the machine type. Supported values are 'pc' or 'q35'.
 
+- `rng0` (object) - TODO
+
+  ```json
+  {
+    "source": "xxx",
+    "max_bytes": xxx,
+    "period": xx
+  }
+  ```
+
+  - `source` (string) - TODO
+
+  - `max_bytes` (int) - TODO
+
+  - `period` (int) - TODO
+
+
 ## Example: Cloud-Init enabled Debian
 
 Here is a basic example creating a Debian 10 server image. This assumes

diff --git a/docs/builders/iso.mdx b/docs/builders/iso.mdx
@@ -399,6 +399,8 @@ or responding to pattern `/dev/.+`. Example:
 
 - `machine` - (string) - Set the machine type. Supported values are 'pc' or 'q35'.
 
+- `rng0` (object) - TODO
+
 ## Boot Command
 
 @include 'packer-plugin-sdk/bootcommand/BootConfig.mdx'