diff --git a/CHANGELOG.md b/CHANGELOG.md index 265ccfade..925b7ab61 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,8 @@ FEATURES: IMPROVEMENTS: * builder/vmware: Workstation 10 support for Linux. [GH-900] +* builder/virtualbox: Support an `export_opts` option which allows + specifying arbitrary arguments when exporting the VM. BUG FIXES: diff --git a/builder/virtualbox/common/export_opts.go b/builder/virtualbox/common/export_opts.go new file mode 100644 index 000000000..1794823f4 --- /dev/null +++ b/builder/virtualbox/common/export_opts.go @@ -0,0 +1,27 @@ +package common + +import ( + "fmt" + "github.com/mitchellh/packer/packer" +) + +type ExportOpts struct { + ExportOpts string `mapstructure:"export_opts"` +} + +func (c *ExportOpts) Prepare(t *packer.ConfigTemplate) []error { + templates := map[string]*string{ + "export_opts": &c.ExportOpts, + } + + errs := make([]error, 0) + for n, ptr := range templates { + var err error + *ptr, err = t.Process(*ptr, nil) + if err != nil { + errs = append(errs, fmt.Errorf("Error processing %s: %s", n, err)) + } + } + + return errs +} diff --git a/builder/virtualbox/common/export_opts_test.go b/builder/virtualbox/common/export_opts_test.go new file mode 100644 index 000000000..b86d0eb40 --- /dev/null +++ b/builder/virtualbox/common/export_opts_test.go @@ -0,0 +1,18 @@ +package common + +import ( + "testing" +) + +func TestExportOptsPrepare_BootWait(t *testing.T) { + var c *ExportOpts + var errs []error + + // Good + c = new(ExportOpts) + c.ExportOpts = "ovf" + errs = c.Prepare(testConfigTemplate(t)) + if len(errs) > 0 { + t.Fatalf("should not have error: %s", errs) + } +} diff --git a/builder/virtualbox/common/step_export.go b/builder/virtualbox/common/step_export.go index c5dc208b2..c17f8cd0b 100644 --- a/builder/virtualbox/common/step_export.go +++ b/builder/virtualbox/common/step_export.go @@ -16,8 +16,9 @@ import ( // Produces: // exportPath string - The path to the resulting export. type StepExport struct { - Format string - OutputDir string + Format string + OutputDir string + ExportOpts string } func (s *StepExport) Run(state multistep.StateBag) multistep.StepAction { @@ -50,6 +51,7 @@ func (s *StepExport) Run(state multistep.StateBag) multistep.StepAction { vmName, "--output", outputPath, + s.ExportOpts, } ui.Say("Exporting virtual machine...") diff --git a/builder/virtualbox/iso/builder.go b/builder/virtualbox/iso/builder.go index 14e5daf65..31660a961 100644 --- a/builder/virtualbox/iso/builder.go +++ b/builder/virtualbox/iso/builder.go @@ -29,6 +29,7 @@ type Builder struct { type config struct { common.PackerConfig `mapstructure:",squash"` vboxcommon.ExportConfig `mapstructure:",squash"` + vboxcommon.ExportOpts `mapstructure:",squash"` vboxcommon.FloppyConfig `mapstructure:",squash"` vboxcommon.OutputConfig `mapstructure:",squash"` vboxcommon.RunConfig `mapstructure:",squash"` @@ -73,6 +74,7 @@ func (b *Builder) Prepare(raws ...interface{}) ([]string, error) { // Accumulate any errors and warnings errs := common.CheckUnusedConfig(md) errs = packer.MultiErrorAppend(errs, b.config.ExportConfig.Prepare(b.config.tpl)...) + errs = packer.MultiErrorAppend(errs, b.config.ExportOpts.Prepare(b.config.tpl)...) errs = packer.MultiErrorAppend(errs, b.config.FloppyConfig.Prepare(b.config.tpl)...) errs = packer.MultiErrorAppend( errs, b.config.OutputConfig.Prepare(b.config.tpl, &b.config.PackerConfig)...) @@ -317,8 +319,9 @@ func (b *Builder) Run(ui packer.Ui, hook packer.Hook, cache packer.Cache) (packe }, new(vboxcommon.StepRemoveDevices), &vboxcommon.StepExport{ - Format: b.config.Format, - OutputDir: b.config.OutputDir, + Format: b.config.Format, + OutputDir: b.config.OutputDir, + ExportOpts: b.config.ExportOpts.ExportOpts, }, } diff --git a/builder/virtualbox/ovf/builder.go b/builder/virtualbox/ovf/builder.go index daf1f0052..f2e6a92f5 100644 --- a/builder/virtualbox/ovf/builder.go +++ b/builder/virtualbox/ovf/builder.go @@ -95,8 +95,9 @@ func (b *Builder) Run(ui packer.Ui, hook packer.Hook, cache packer.Cache) (packe }, new(vboxcommon.StepRemoveDevices), &vboxcommon.StepExport{ - Format: b.config.Format, - OutputDir: b.config.OutputDir, + Format: b.config.Format, + OutputDir: b.config.OutputDir, + ExportOpts: b.config.ExportOpts.ExportOpts, }, } diff --git a/builder/virtualbox/ovf/config.go b/builder/virtualbox/ovf/config.go index 56f37f43f..054d09e3a 100644 --- a/builder/virtualbox/ovf/config.go +++ b/builder/virtualbox/ovf/config.go @@ -13,6 +13,7 @@ import ( type Config struct { common.PackerConfig `mapstructure:",squash"` vboxcommon.ExportConfig `mapstructure:",squash"` + vboxcommon.ExportOpts `mapstructure:",squash"` vboxcommon.FloppyConfig `mapstructure:",squash"` vboxcommon.OutputConfig `mapstructure:",squash"` vboxcommon.RunConfig `mapstructure:",squash"` @@ -49,6 +50,7 @@ func NewConfig(raws ...interface{}) (*Config, []string, error) { // Prepare the errors errs := common.CheckUnusedConfig(md) errs = packer.MultiErrorAppend(errs, c.ExportConfig.Prepare(c.tpl)...) + errs = packer.MultiErrorAppend(errs, c.ExportOpts.Prepare(c.tpl)...) errs = packer.MultiErrorAppend(errs, c.FloppyConfig.Prepare(c.tpl)...) errs = packer.MultiErrorAppend(errs, c.OutputConfig.Prepare(c.tpl, &c.PackerConfig)...) errs = packer.MultiErrorAppend(errs, c.RunConfig.Prepare(c.tpl)...) diff --git a/website/source/docs/builders/virtualbox-iso.html.markdown b/website/source/docs/builders/virtualbox-iso.html.markdown index 490fbeb65..2f417dca0 100644 --- a/website/source/docs/builders/virtualbox-iso.html.markdown +++ b/website/source/docs/builders/virtualbox-iso.html.markdown @@ -211,6 +211,10 @@ Optional: machine, without the file extension. By default this is "packer-BUILDNAME", where "BUILDNAME" is the name of the build. +* `export_opts` (string) - Additional options to pass to the `VBoxManage export`. + This can be useful for passing product information to include in the resulting + appliance file. + ## Boot Command The `boot_command` configuration is very important: it specifies the keys diff --git a/website/source/docs/builders/virtualbox-ovf.html.markdown b/website/source/docs/builders/virtualbox-ovf.html.markdown index 2112e276d..47378e6c2 100644 --- a/website/source/docs/builders/virtualbox-ovf.html.markdown +++ b/website/source/docs/builders/virtualbox-ovf.html.markdown @@ -151,6 +151,10 @@ Optional: This can be useful for passing "keepallmacs" or "keepnatmacs" options for existing ovf images. +* `export_opts` (string) - Additional options to pass to the `VBoxManage export`. + This can be useful for passing product information to include in the resulting + appliance file. + ## Guest Additions Packer will automatically download the proper guest additions for the