259 Commits (70aa0516deb7cd0d03d321fa025fa5ae581bb883)

Author	SHA1	Message	Date
Adrien Delorme	a36b94320a	virtualbox: simplify ParseSnapshotData	6 years ago
Adrien Delorme	116a038de9	Update snapshot.go ... avoid a panic on a too short slice.	6 years ago
Adrien Delorme	14243ce56a	Merge pull request #8305 from hashicorp/fix_8301 ... allow nvme, since virtualbox 6.0 allows it	6 years ago
Megan Marsh	6f54468dd5	fix mocks	6 years ago
Megan Marsh	9343a23021	add driver calls to enable nvme controller	6 years ago
Adrien Delorme	819329228a	Change back to make sure all durations are a time.Duration ... It is simply the best/simplest solution and trying to prevent users from passing and integer here would be like opening a can of worms. Because: * we cannot make mapstructure validate our duration string ( with an UnmarshalJSON func etc.) * we cannot make mapstructure spit a string instead of a duration and packer will decode-encode-decode config. * the hcl2 generated code asks for a string, so this will be enforced by default.	6 years ago
Adrien Delorme	bf3d9841c6	Force durations to be passed a strings ... Before this commit it was possible to set a duration using an integer or a float. Go's time.Duration is an int64 internally an mapstructure will take advantage of this and load the number as a int64 but `1` means one ns which is unexpected/confusing. To avoid confusion and enforce readability this forces users to pass a string with a unit for a duration; ex "56s".	6 years ago
Adrien Delorme	3fe9d52e3f	Merge remote-tracking branch 'origin/master' into scrape_doc_to_builder_struct_config	7 years ago
Brian Fox	5b6dcb0c94	fix(virtualbox) LoadSnapshots should succeed even if VM has no snapshots	7 years ago
Adrien Delorme	597a3e323b	Merge remote-tracking branch 'origin/master' into scrape_doc_to_builder_struct_config	7 years ago
Adrien Delorme	350aa54274	Update step_shutdown_test.go ... bump test duration	7 years ago
Adrien Delorme	4cb7c30987	Merge remote-tracking branch 'origin/master' into scrape_doc_to_builder_struct_config	7 years ago
Megan Marsh	02399f32f6	dont bother with a targetpath for the vbox additions; use a hash like everything else	7 years ago
Thomas Meckel	d9b8623f64	StepShutdown now always waits for the shutdown of the virtual machine to complete, not only if a shutdown command is specified	7 years ago
Thomas Meckel	b873876670	Added test TestSnapshot_EnsureParents ... No using asserts NotNil and Nil	7 years ago
Thomas Meckel	6fd0b3e83e	Fixed a bug in ParseSnapshotData ... Added fuction GetSnapshots	7 years ago
Thomas Meckel	225a140055	Applied correct source format (got fmt) to builder\virtualbox\common\driver_mock.go	7 years ago
Thomas Meckel	51e5ae69db	Corrected method signatures in builder\virtualbox\common\driver_mock.go	7 years ago
Thomas Meckel	ab6a163ff2	Added missing newline at end of builder\virtualbox\common\driver_mock.go	7 years ago
Thomas Meckel	e9b5332e99	Added missing functions to builder\virtualbox\common\driver_mock.go	7 years ago
Thomas Meckel	fd5a2db58f	Removed commented (obsolete) code from builder\virtualbox\common\driver_4_2.go	7 years ago
Thomas Meckel	7d3c84e5f9	Changed VirtualBox VM builder according to current builder interface	7 years ago
Thomas Meckel	2ef911f6f0	Fixed bugs in snapshot and builder code	7 years ago
Thomas Meckel	ebd2780e5d	implemented first unit tests for snapshot.go	7 years ago
Thomas Meckel	f1f7c4e10c	* added function ParseSnapshotData to parse a string representation of a VBox snapshot tree ... * fixed bugs	7 years ago
Thomas Meckel	45e2095ae3	Moved parsing of tree string representation to new function in snapshot.go	7 years ago
Thomas Meckel	092e32fe9e	First version of reworked snapshot implementation	7 years ago
Thomas Meckel	a6074894f1	Defined struct and first function for enhanced snapshot handling	7 years ago
Thomas Meckel	d3202497ae	First working version of virtualbox/vm builder	7 years ago
Thomas Meckel	2a531f8ad6	Merged changes from local branch	7 years ago
Adrien Delorme	74f0d56cfc	virtualbox merge vbpx manage configs	7 years ago
Adrien Delorme	01eff9472a	virtualbox: merge ExportOpts and ExportConfig structs	7 years ago
Adrien Delorme	c11ef90cb6	use interpolate.NewContext() instead of testConfigTemplate(t) ... because it's what's happenning there	7 years ago
Adrien Delorme	4bf6a56d71	split communitacor.SSH & communitacor.WinRM to facilitate documentation ... put doc for communicator into the struct	7 years ago
Adrien Delorme	4399684372	make fmt autogenerated docs	7 years ago
Adrien Delorme	e6cbb013ba	add // go:generate struct-markdown to all previously edited files	7 years ago
Adrien Delorme	f1917edd34	generate the comments for config struct of builders scraping doc website	7 years ago
Adrien Delorme	d72040f4fa	move retry code into the common/retry pkg and make retry context aware	7 years ago
Adrien Delorme	f555e7a9f2	allow a provisioner to timeout ... * I had to contextualise Communicator.Start and RemoteCmd.StartWithUi NOTE: Communicator.Start starts a RemoteCmd but RemoteCmd.StartWithUi will run the cmd and wait for a return, so I renamed StartWithUi to RunWithUi so that the intent is clearer. Ideally in the future RunWithUi will be named back to StartWithUi and the exit status or wait funcs of the command will allow to wait for a return. If you do so please read carrefully https://golang.org/pkg/os/exec/#Cmd.Stdout to avoid a deadlock * cmd.ExitStatus to cmd.ExitStatus() is now blocking to avoid race conditions * also had to simplify StartWithUi	7 years ago
Adrien Delorme	c4f3dccc14	rename interpolation context from ctx to ictx and contexts to ctx to avoid conflicts	7 years ago
Adrien Delorme	a4bf94dd3c	change Builder to be passed a context for cancellation ... we have to to give it to our hook	7 years ago
Megan Marsh	9f1a4e0fed	Merge pull request #7423 from hashicorp/lock_ports ... Lock packer ports using a lock file	7 years ago
Adrien Delorme	5a6dffde9a	use port as ints	7 years ago
Adrien Delorme	c214f6735b	make everything a uint	7 years ago
Adrien Delorme	f67b5ff59d	virtualbox StepConfigureVRDP: use common/net pkg to find open port	7 years ago
Adrien Delorme	59370986fb	virtualbox StepForwardSSH: use common/net pkg to find open port	7 years ago
Vadym Haidamaka	414d0750f0	Add tests to validate 'none' communicator in the virtualbox builder	7 years ago
Vadym Haidamaka	0263503c45	Validate virtualbox_version_file and guest_additions_mode when communicator is none	7 years ago
Adrien Delorme	9f82b75e57	Use the hashicorp/go-getter to download files ... * removed packer.Cache and references since packer.Cache is never used except in the download step. The download step now uses the new func packer.CachePath(targetPath) for this, the behavior is the same. * removed download code from packer that was reimplemented into the go-getter library: progress bar, http download restart, checksuming from file, skip already downloaded files, symlinking, make a download cancellable by context. * on windows if packer is running without symlinking rights and we are getting a local file, the file will be copied instead to avoid errors. * added unit tests for step_download that are now CI tested on windows, mac & linux. * files are now downloaded under cache dir `sha1(filename + "?checksum=" + checksum) + file_extension` * since the output dir is based on the source url and the checksum, when the checksum fails, the file is auto deleted. * a download file is protected and locked by a file lock, * updated docs * updated go modules and vendors	7 years ago
Megan Marsh	3dc1dafe58	Merge pull request #7287 from stephen-fox/ephemeral-ssh-key-pair-issue-7225 ... virtualbox: create ephemeral SSH key pair for build process	7 years ago