Commit Graph

160 Commits

Author SHA1 Message Date
Matthew Hooker f1175c1921
parallels/vmware: don't wait for shutdown command.
Resolves #4134

replaces/ closes #4379

Leaving Hyper-V builder alone for now until we can get a case that reproduces.
2017-01-20 01:44:18 -08:00
Matthew Hooker d2e59e4e92
set PACKER_HTTP_ADDR env var when available.
If using a builder that has an http server set up for file transfer,
expose the connection info to the shell provisioner through the environment
variable PACKER_HTTP_ADDR.

Closes #2869
2017-01-16 23:19:52 -08:00
Rickard von Essen 85cf00069f Merge pull request #4294 from legal90/iso-target-extension
Add "iso_target_extension" option for all local *-iso builders
2017-01-04 13:48:28 +01:00
Mikhail Zholobov 57d07f227a
Add "iso_target_extension" option for all local *-iso builders
This option allows to set the extension of the ISO file after download.
Defaults to "iso". It makes sense for building Mac OS X guests, where the
bootable image is actually a DMG, not an ISO.
In particular, it is important for "parallels-iso" builder to set the right extension.
2016-12-17 12:50:30 +02:00
Mikhail Zholobov aa623e22a3
golint: Add comments for exported methods 2016-12-17 00:37:04 +02:00
Mikhail Zholobov 73d87b07de
golint: Fix lint offences 2016-12-17 00:37:03 +02:00
Mikhail Zholobov 00a90e622b
Remove unused constant "KeyLeftShift" 2016-12-17 00:37:03 +02:00
Mikhail Zholobov 1ca841b499
Fix shadowed declarations
Performed by "go vet" tool
2016-12-17 00:37:03 +02:00
Mikhail Zholobov dde445bd0d
Fix style: Organize imports in groups
Performed by "goimports" tool
Style conventions: https://github.com/golang/go/wiki/CodeReviewComments#imports
2016-12-17 00:37:03 +02:00
Mikhail Zholobov 319b85cb7a
Fix style: Use a consistent case for initialisms
https://github.com/golang/go/wiki/CodeReviewComments#initialisms
2016-12-17 00:36:56 +02:00
Rickard von Essen 96e9a8e6e9 Removed default value for ssh_username 2016-11-17 22:30:34 +01:00
Matthew Hooker d920b3fbf4 run gofmt 2016-11-01 14:08:04 -07:00
Rickard von Essen 3c8dabba9e Fixed formatting 2016-10-11 23:43:50 +02:00
Rickard von Essen 5e96709ee9 Merge pull request #2919 from arizvisa/floppy-recurse
Added an option for copying entire subdirectories via floppy_dirs (supplants floppy_files)
2016-10-08 16:51:16 +02:00
Ali Rizvi-Santiago 86c00490e9 Renamed any and all instances of the word "FloppyContents" to "FloppyDirectories".
Ensure that all builders include FloppyDirectories in the StepCreateFloppy options.
Changed the way the unit-tests in common/step_create_floppy_test work to use the static test-fixtures directory instead of creating the paths dynamically.
Removed a duplicate line of documentation from parallels-pvm.html.md that occurred during rebasing.
2016-09-27 23:31:42 -05:00
Rickard von Essen 06e3539ffa Merge pull request #3767 from taliesins/ExtendScanCodes
Add support for ctrl, shift and alt keys and using them as key modifier.
2016-09-20 21:36:46 +02:00
Orivej Desh 6762965696 Add -on-error command line argument to allow preserving artifacts on builder errors
Resolves #409
2016-09-16 12:15:00 +00:00
Taliesin Sisson a87ad05866 Add support for ctrl, shift and alt keys.
Add support for using ctrl, shift and alt as key modifiers. So you can now achieve ctrl+c by using "<leftCtrlOn>c<leftCtrlOff>".

Updated documentation for new key stroke tokens.
2016-07-31 19:05:10 +01:00
Ricard Clau acededfc6e tests actually test the floppies 2016-07-27 21:59:21 +01:00
Rickard von Essen fb00d07d20 Support future versions of Parallels by using the latest driver
If the Parallels Desktop for Mac version is higher than the latest
Packer driver use the latest available driver. Before this Packer needed
to be updated with every new version of PD even if there was no change
to the driver.
2016-07-02 22:18:54 +02:00
Sean Chittenden ef42021f8f Fix unit tests, broken in #3549. (#3548) 2016-05-17 15:42:15 -07:00
Sean Chittenden d0fd698fb5
Pause between boot_commands when debugging a Parallels build
When debugging a build (or maintaining an existing packer file), teach `packer build -debug` how to step through individual `boot_command`s in order to triage the packer template.
2016-05-17 17:10:10 -04:00
Rickard von Essen 46f74d8347 Parallels: Support relative paths in output_directory
Fixes #3238
2016-02-21 16:52:51 +01:00
James Bishopp bd8fb014c4 Removed ssh_key_path
- removed backwards compatibility code
- ensured key usage came from SSHPrivateKey configuration
- changed tests to use private_key
2016-02-12 17:24:42 -08:00
Rickard von Essen fd6282529c Remove deprecated parallels_tools_host_path and guest_os_distribution 2016-02-01 12:25:01 +01:00
Mark Peek 7f149e595d Refactor http server config into common 2015-11-01 14:45:47 -08:00
Mark Peek cdcffecc2d Refactor builder ISO options
The ISO builders (parallels, qemu, virtualbox, and vmware) had too
much common code which needed to be maintained separately. This change
moves that code to a common ISO configuration.
2015-10-20 16:27:47 -07:00
Mark Peek e9657e6a9f Merge pull request #2642 from oliviertremblay/master
Make TargetPath customizable
2015-10-20 12:11:13 -07:00
Rickard von Essen 4015c1789b Fixes #2836 - Use "/Applications/Parallels Desktop.app" as fallback app path. 2015-10-16 19:57:39 +02:00
Rickard von Essen 141dcd5c03 Fixes #2828: parallels prlctl_post don't interpolate values. 2015-10-15 09:28:29 +02:00
Chris Bednarski f01956405b Merge pull request #2654 from mitchellh/b-xmlpath
Update dependency to an SSL-enabled path; this was migrated from launchpad to github
2015-10-12 16:39:35 -07:00
Rickard von Essen 3f14b9d984 Merge pull request #2731 from legal90/prl-compact-disk
Parallels: Add "CompactDisk" build step
2015-09-16 14:03:31 +02:00
Mikhail Zholobov abfe706fc6 builder/parallels: Add "StepCompactDisk" 2015-09-11 11:38:20 +03:00
Mikhail Zholobov f7b26e44fe builder/parallels: Add "CompactDisk" driver function
This function compacts the specified virtual disk image.
2015-09-11 11:37:52 +03:00
Mikhail Zholobov 4ebee7bf3f builder/parallels: Add "DiskPath" driver function
This function determines path to the first virtual disk image of the specified virtual machine.
2015-09-05 13:08:24 +03:00
Rickard von Essen 83980d2326 Enable headless mode by default on Parallels Desktop 11 2015-08-24 15:09:29 +02:00
Rickard von Essen 7d3afc882a Add check for Parallels Desktop edition in PD 11.
Starting since Parallels Desktop 11, the command line functionality is
available only in Pro and Business editions.
2015-08-22 13:15:59 +02:00
Chris Bednarski 0a79546cf9 Update dependency to an SSL-enabled path; this was migrated from launchpad to github 2015-08-21 17:17:56 -07:00
Olivier Tremblay f301a6454d Added documentation, renamed config param to iso_target_path 2015-08-20 07:37:24 -04:00
Olivier Tremblay 661552dfd5 Merge branch 'master' into custom-targetpath 2015-08-20 07:26:22 -04:00
Rickard von Essen da82ff3fd6 Fix interpolation of {{.Flavor}} in parallels_tools_guest_path. (2)
Actually fix the error...

Fixes [GH-2543]
2015-08-12 10:28:06 +02:00
Rickard von Essen 5a6bcdeb78 Fix interpolation of {{.Flavor}} in parallels_tools_guest_path.
Fixes [GH-2543]
2015-08-12 01:34:08 +02:00
Olivier Tremblay 223e35fc65 Makes StepDownload's TargetPath customizable.
I exposed TargetPath as a config file option "target_path". I don't like
the name, but it follows the naming convention. The purpose of TargetPath
stands unmodified, and it enables a fair amount of customization.
2015-07-28 07:45:02 -04:00
Mitchell Hashimoto 1aa1399e85 builder/parallels: fix compilation issuse 2015-06-22 12:46:13 -07:00
Mitchell Hashimoto 13c2c4660d Merge branch 'issue_2080' of https://github.com/rickard-von-essen/packer into rickard-von-essen-issue_2080 2015-06-22 12:39:30 -07:00
Mitchell Hashimoto 84189f7a28 builder/*: properly save interpolation context 2015-06-22 09:22:42 -07:00
Mikhail Zholobov 0a53fbc29d builder/parallels: Add mock for "SetDefaultConfiguration" method
Fixes unit test failures
2015-06-18 14:36:32 +03:00
Mikhail Zholobov 2860bfdf82 builder/parallels: Apply default settings on the VM creation step. 2015-06-18 12:08:17 +03:00
Mikhail Zholobov f7af571cd9 builder/parallels: Add "SetDefaultConfiguration" function
This functions applies the default configuration to the virtual machine. Also, it disables some integration features
which should not present in the resulted VM image.

Functions are different in PD 9 and 10 structs because some additional options appeared only in Parallels Desktop 10 release.
2015-06-18 12:04:02 +03:00
Mitchell Hashimoto 115d583cff helper/communicator: make host more generic 2015-06-13 19:23:33 -04:00
Mitchell Hashimoto 820bad6939 builder/parallels: convert to new comm type 2015-06-13 18:43:27 -04:00
Mark Peek f398352996 Fix a bug where interpolation was broken in some builders 2015-06-12 14:00:59 -07:00
Rickard von Essen 8393b85ed5 Support Parallels Desktop 11 2015-06-10 07:41:12 +02:00
Mitchell Hashimoto 23a48d6619 go fmt 2015-06-08 21:34:20 -07:00
jszwedko b1497b951c code.google.com/p/go.crypto/ssh -> golang.org/x/crypto/ssh
code.google.com/p/go.crypto/ssh is now at golang.org/x/crypto/ssh as of
https://code.google.com/p/go/source/detail?spec=svn.crypto.69e2a90ed92d03812364aeb947b7068dc42e561e&repo=crypto&r=8fec09c61d5d66f460d227fd1df3473d7e015bc6

Using the code.google.com import redirects properly, but runs into
issues if you try to use a subpackage of `ssh`, e.g. `agent` which
refers to golang.org/x/crypto/ssh causing conflicts if your types expect
code.google.com/p/go.crypto/ssh.

This is a precursor to a PR for #1066.
2015-05-28 08:17:49 -07:00
Mitchell Hashimoto 4bb16ac223 fix failing tests 2015-05-27 14:30:59 -07:00
Mitchell Hashimoto b2b74431ec builder/parallels/pvm: interpolation 2015-05-27 13:51:24 -07:00
Mitchell Hashimoto 76c2d2cb0e builder/parallels/iso: new interpolation 2015-05-27 13:49:31 -07:00
Rickard von Essen 1365627e31 Fixes #2080 Add prlctl_post in builder Parallels
This adds config option prlctl_post for builder parallels-iso/pvm. It
allows additional prlctl commands to run after the VM has been shutdown
just before being exported.
2015-05-03 11:18:48 +02:00
Emil Hessman b19f9b9777 builder/parallels/iso: fix incorrect printf verb type
Fixes the following vet report:

builder/parallels/iso/builder_test.go:78: arg b.config.DiskSize for printf verb %s of wrong type: uint
2015-02-25 05:43:17 +01:00
Rickard von Essen d424ed1e37 builder/parallels: Use most recent DHCP lease when determening IP.
Fixes #1746. Problem establishing SSH connection with Fedora 21 and
Parallels
2014-12-28 22:22:16 +01:00
Mikhail Zholobov a5b21c59ba builder/parallels: Move setting the boot order to the separated step 2014-11-27 17:13:50 +03:00
Mikhail Zholobov 05affceaef builder/parallels: Attach bootable ISO exactly to cdrom0 [GH-1667] 2014-11-27 17:11:16 +03:00
Mitchell Hashimoto 0db6cd3533 Merge pull request #1330 from qur/qemu-vagrant
Qemu vagrant
2014-10-28 08:42:17 -07:00
Mikhail Zholobov 6e778e7244 builder/parallels: Default cdrom0 should be disconnected
Otherwise, the error message will be displayed in GUI:
"Unable to connect CD/DVD 0"
2014-10-21 16:44:43 +04:00
Rickard von Essen d75d44ad9e Merge pull request #1548 from legal90/create_disk
builder/parallels: Create VM without hdd and then add it later
2014-10-04 23:06:15 +02:00
Mikhail Zholobov eb37742ab0 builder/parallels: Disable default cdrom0 device and then enable it on cleanup
This is needed to make 'cdrom0' device unavailable during the ISO installation process.
It is required for some guest OS types, such as FreeBSD and OmniOS.
2014-10-04 22:39:54 +02:00
Mikhail Zholobov 13251ced49 builder/parallels: Added setting the boot order
CD/DVD drive with installation ISO should be exactly the 2nd device in the VM boot order.
This ensures that VM will boot from this ISO at the first boot time only.
2014-10-04 22:39:54 +02:00
Mikhail Zholobov 280c3c52de builder/parallels: Attach ISO images to the separate cdrom device
For each ISO image the individual cdrom device will be added to the VM. During the cleanup these devices will be deleted.
It makes attach steps more clear - there is no doubt what is the name of the device.

Related to: [mitchellh/packer#1502]
2014-10-04 22:39:54 +02:00
Mikhail Zholobov 0cfe58193f builder/parallels: Added driver method 'DeviceAddCdRom' 2014-10-04 22:39:54 +02:00
JessThrysoee 5394ce61fd builder/parallels: Rely on Cleanup functions to detach devices
Remove the redundant StepRemoveDevices and rely on the cleanup to be handled by:
* StepAttachParallelsTools.Cleanup
* StepAttachFloppy.Cleanup
* stepAttachISO.Cleanup
2014-10-04 22:39:54 +02:00
Rickard von Essen 57ad64da30 Merge pull request #1501 from JessThrysoee/current_fdd0_does_not_exist
builder/parallels: Ignore 'The fdd0 device does not exist'
2014-10-04 18:04:08 +02:00
Mikhail Zholobov 48c64b05cb builder/parallels: Init VM without hdd and attach it later
Avoid the disk resize because it is unsupported for some guests (OS X)
http://kb.parallels.com/en/113653
2014-10-01 16:13:46 +04:00
Julian Phillips 90a57c411f Expand Artifact API to expose build state
In order that something consuming an artifact can have access to extra
builder specific data add the State method which allows the caller to
ask for arbitary values by name.
2014-09-22 11:15:47 +01:00
JessThrysoee 16b82d6d15 builder/parallels: Bundle python version of prltype
Uses the Python API from Parallels Virtualization SDK to write
boot commands.

This eliminates a 3rd party requirement and makes it easier for people
not using homebrew to get started with packer.
2014-09-19 22:19:15 +02:00
JessThrysoee cf6532ffe2 builder/parallels: Ignore 'The fdd0 device does not exist'
A current floppy disk probably doesn't exist, so the build shouldn't
fail when we try to delete it.
2014-09-18 20:36:05 +02:00
Mitchell Hashimoto 75395af12d Merge branch '1064-fix-upload-file-permissions' of github.com:rasa/packer into rasa-1064-fix-upload-file-permissions
Conflicts:
	builder/parallels/common/step_upload_parallels_tools.go
	builder/vmware/common/step_upload_tools.go
	provisioner/chef-client/provisioner.go
	provisioner/chef-solo/provisioner.go
2014-09-08 13:20:46 -07:00
Rickard von Essen 365b44c5af parallels-iso: allow absolute output directory.
Fixes [GH-1466]
2014-09-08 11:00:59 +02:00
Rickard von Essen 5ad7737d55 parallels-pvm: Optionally assign new MAC addresses to imported VMs [GH-1404]
New config option:
reassign_mac (boolean) - If this is "false" the MAC address of the first
NIC will reused when imported else a new MAC address will be generated by
Parallels. Defaults to "false".
2014-09-05 11:33:28 +02:00
Mitchell Hashimoto 76a8221636 builder/*: extract key path to ssh.Signer 2014-09-03 20:23:39 -07:00
Mikhail Zholobov 16cb6f60c8 builder/parallels: "guest_os_distribution" renamed to "guest_os_type".
"guest_os_distribution" become deprecated and is not required anymore.
But if it is defined (in outdated templates),then the value will be copied to "guest_os_type" and warning will be displayed.
2014-09-02 18:14:50 +04:00
Mikhail Zholobov bed6270288 builder/parallels: Detect path to Parallels Tools ISO automatically.
'parallels_tools_host_path' param is deprecated
'parallels_tools_flavor' is added (mandatory).
2014-09-02 18:14:26 +04:00
Mikhail Zholobov 40c169f757 builder/parallels: Added 'StepPrepareParallelsTools' 2014-09-02 18:14:26 +04:00
Mikhail Zholobov 13b355eaf0 builder/parallels: Added 'ToolsConfig' struct 2014-09-02 18:13:40 +04:00
Mikhail Zholobov 909d5ccb54 builder/parallels: 'ToolsIsoPath' added 2014-09-02 11:02:03 +04:00
Rickard von Essen f9cddd2e18 builder/parallels: Added some navigation keys. Fixes [GH-1442] 2014-08-26 01:00:58 +02:00
Mikhail Zholobov fc2d695b0f builder/parallels: Added compatibility with Parallels Desktop 10 2014-08-24 07:38:24 +04:00
Mikhail Zholobov 10be7e27de builder/parallels: Added platform check 2014-08-24 07:30:29 +04:00
Mikhail Zholobov 9ad2b98673 builder/parallels: Removed config parameter 'ostype'
We have to set either 'ostype' or 'distribution', but not both.
2014-08-24 07:30:29 +04:00
Rickard von Essen 732c767bfd parallels: Support for Parallels Desktop 10 2014-08-23 20:23:13 +02:00
Rickard von Essen fe37d9b6e1 parallels-iso: ISO not removed from VM after install [GH-1338]
Also remove floppy and prl-tools ISO if applicable.
2014-08-08 10:07:32 +02:00
Rickard von Essen 25d2a74f49 Merge pull request #1187 from rickard-von-essen/fix_typing
builder/parallels: Fixed problems with character when typing boot command
2014-06-22 21:05:40 +02:00
Rickard von Essen a5d39cebc7 Merge pull request #1173 from rickard-von-essen/pvm_bootcommand
[feature] Support boot command for pvm builder
2014-06-22 21:05:04 +02:00
Rickard von Essen 3f3deab162 Merge pull request #1279 from rickard-von-essen/issue_1226
Parallels-iso errors while removing floppy drive GH-1226
2014-06-22 21:04:02 +02:00
Rickard von Essen 905f242582 Parallels-iso errors while removing floppy drive GH-1226
Status "stopping" should be handled in IsRunning
2014-06-06 15:59:27 +02:00
Paul Palmer 5a8f56b6ad Parallels errors while creating floppy disk GH-1225
Patch applied with modifications - tests updated to pass.
2014-06-06 15:54:19 +02:00
Rickard von Essen 29344a5631 Parallels builder mistypes boot command on OmniOS
Fixes [GH-1185] and adds tests for the boot command.
Also requires the latest prl-utils (e592a15) to fix the stuck on enter
while typing.
2014-05-19 20:14:50 +02:00
Rickard von Essen 4d994deb3a [feature] Support boot command for pvm builder
Similar to [GH-1082] but for parallels-pvm builder.
2014-05-19 20:04:13 +02:00
Rickard von Essen 4cdce51df5 builder/parallels: Don't delete sound, since it doesn't allways exist. 2014-05-11 11:20:56 +02:00
Rickard von Essen bd25866e90 builder/parallels: Select which interface to connect to.
The IP address that the guest OS connect to download files from the
http_dir is determined by searching for a host interface with an IP
assigned. Now it is possible to use host_interfaces to select which
interfaces are searched.
2014-05-11 11:12:50 +02:00