578 Commits (9891e75f75696d514baff262e8aacd9eb25a8051)

Author	SHA1	Message	Date
William Brooks	31622b50ac	applied fmt	6 years ago
William Brooks	55ae803852	Add Hyper-V support for Gen-1 boot order with ISO	6 years ago
Chris Cunningham	e94ed9a683	update the note about Windows symlinks ... GH-7534 and GH-7545 disabled the use of symlinks for copying local ISOs, per: https://github.com/hashicorp/packer/blob/master/common/step_download.go#L106 Update the documentation to match, as without reading the source it is implied that this should work.	6 years ago
Megan Marsh	dde7302265	Merge pull request #8689 from hashicorp/fix_8530 ... Add iso file name when targetPath is a directory	6 years ago
Moss	f2c8e8491c	Fix format	6 years ago
Moss	e020da49f3	Add iso file name when targetPath is absolute	6 years ago
Wilken Rivera	561f02cc2f	builder/azure-arm: Update logic for setting subscriptionID (#8685) ... * builder/azure-arm: Update logic for setting subscriptionID Previously, when using managed identities, the Azure builder would set the SubscriptionID in the Prepare method. But would not update it after getting the updated SubscriptionID from the metadata server. This change updates the Run method to ensure a valid subscriptionID is saved to the statebag before continuing with an image build. Co-authored-by: Paul Meyer <paul.meyer@outlook.com>	6 years ago
Sylvia Moss	dc31bad539	Sharing info with post-processors via artifact (#8632)	6 years ago
Moss	0566f1f999	Extract http ip discover to a new step	6 years ago
Sylvia Moss	0b7251a4bb	Share .Device and .MountPath between builders, provisioners and post-processors (#8621)	6 years ago
Sylvia Moss	0677b02e18	Share SourceImageName with provisioners and manifest post-processor (#8603)	6 years ago
Megan Marsh	90bdcf58bd	update step_download to return an ActionContinue if the URls field is empty. this allows us to simplify the hyperv builder, and is still safe because all other builders and uses of step_download already validate that the iso url is not empty if that's what they need, most of them inside of the IsoConfig prepare function.	6 years ago
Adrien Delorme	4b7132c87c	Merge remote-tracking branch 'origin/master' into sharing_info	6 years ago
Adrien Delorme	0785c2f6fc	build using HCL2 (#8423) ... This follows #8232 which added the code to generate the code required to parse HCL files for each packer component. All old config files of packer will keep on working the same. Packer takes one argument. When a directory is passed, all files in the folder with a name ending with “.pkr.hcl” or “.pkr.json” will be parsed using the HCL2 format. When a file ending with “.pkr.hcl” or “.pkr.json” is passed it will be parsed using the HCL2 format. For every other case; the old packer style will be used. ## 1. the hcl2template pkg can create a packer.Build from a set of HCL (v2) files I had to make the packer.coreBuild (which is our one and only packer.Build ) a public struct with public fields ## 2. Components interfaces get a new ConfigSpec Method to read a file from an HCL file. This is a breaking change for packer plugins. a packer component can be a: builder/provisioner/post-processor each component interface now gets a `ConfigSpec() hcldec.ObjectSpec` which allows packer to tell what is the layout of the hcl2 config meant to configure that specific component. This ObjectSpec is sent through the wire (RPC) and a cty.Value is now sent through the already existing configuration entrypoints: Provisioner.Prepare(raws ...interface{}) error Builder.Prepare(raws ...interface{}) ([]string, error) PostProcessor.Configure(raws ...interface{}) error close #1768 Example hcl files: ```hcl // file amazon-ebs-kms-key/run.pkr.hcl build { sources = [ "source.amazon-ebs.first", ] provisioner "shell" { inline = [ "sleep 5" ] } post-processor "shell-local" { inline = [ "sleep 5" ] } } // amazon-ebs-kms-key/source.pkr.hcl source "amazon-ebs" "first" { ami_name = "hcl2-test" region = "us-east-1" instance_type = "t2.micro" kms_key_id = "c729958f-c6ba-44cd-ab39-35ab68ce0a6c" encrypt_boot = true source_ami_filter { filters { virtualization-type = "hvm" name = "amzn-ami-hvm-????.??.?.????????-x86_64-gp2" root-device-type = "ebs" } most_recent = true owners = ["amazon"] } launch_block_device_mappings { device_name = "/dev/xvda" volume_size = 20 volume_type = "gp2" delete_on_termination = "true" } launch_block_device_mappings { device_name = "/dev/xvdf" volume_size = 500 volume_type = "gp2" delete_on_termination = true encrypted = true } ami_regions = ["eu-central-1"] run_tags { Name = "packer-solr-something" stack-name = "DevOps Tools" } communicator = "ssh" ssh_pty = true ssh_username = "ec2-user" associate_public_ip_address = true } ```	6 years ago
Megan Marsh	0f6d1beccf	add an extra string array to the Prepare() return values in the builder interfaces; this sets up the ability for builders to give the provisioners custom user-accessible build-time variables.	6 years ago
Megan Marsh	21a849354b	fix hook data loading	6 years ago
Megan Marsh	a125f81fba	add some more comments	6 years ago
Megan Marsh	82367a88f8	reorganize placeholder data call to live with provisioner implementation; force users to use the generated function, therefore forcing validation, for all variables except winrmpassword, by doing a simple string check against the placeholder data.	6 years ago
Megan Marsh	ac570e0cc0	store ssh public and private keys on shared info	6 years ago
Megan Marsh	88d4ce37c9	template functioin to let Provisioners access PACKER_RUN_UUID	6 years ago
Megan Marsh	84063d2132	implement ID and Type values in the provisioner template info sharing	6 years ago
Megan Marsh	e1652696a6	Rather than using reflection to grab the whole SSH config, for now just use the config convenience functions to get the majority of needed values	6 years ago
Megan Marsh	3389d843f0	update all provisioners that used the winrmpassword tooling to use the new generateddata option	6 years ago
Megan Marsh	39fd462b56	change all provision func signatures to use map[string]interface{}	6 years ago
Megan Marsh	5d266b20d0	generated func now validates based on data provided in context, which needs to include the future values if it is to be interpolated properly.	6 years ago
Megan Marsh	528b853178	stub out generated func for validation	6 years ago
Megan Marsh	7616fbcf3f	skip shell local for now	6 years ago
Megan Marsh	6f418d0e54	get data sharing to a working state with the powershell provisioner	6 years ago
Megan Marsh	f4c3501af5	pass struct of generated data into provision() call	6 years ago
Sylvia Moss	096cab9b48	Fix floppy doc markdown typo (#8458)	6 years ago
Megan Marsh	76bfaf9da8	Merge pull request #8394 from hashicorp/fix_iso_target_path ... Don't cache when TargetPath is non-nil	6 years ago
Megan Marsh	a8ffa60b46	remove the already-included packer config from top level struct	6 years ago
Megan Marsh	7d4353864f	use shell struct in shell-local provisioner and post-processor.	6 years ago
Megan Marsh	cf0130b9ed	pick up dropped error	6 years ago
Megan Marsh	9fc8ed02ae	Don't cache when TargetPath is non-nil; this messes up use of the iso_target_path iso template option.	6 years ago
Megan Marsh	ae184fe31d	ask for VM by name to reduce looping.	6 years ago
nywilken	3522e13b1e	provisioner/shell: Add `env_var_format` support to shell provisioner ... * Move setting of default EnvVarFormat to the Prepare function * Add generated hcl2 structure for the updated Shell provisioner config * Move `EnvVarFormat` field to shared common shell type * Add description for the `env_var_format` parameter * Update `env_var_format` when used with `use_env_var_file`	6 years ago
Adrien Delorme	c0db000dc3	Merge pull request #8310 from alrs/resuscitate-common-floppy-tests ... common: Resuscitate Floppy Tests	6 years ago
Lars Lehtonen	5926997a31	common: remove unused test function getDirectory()	6 years ago
Lars Lehtonen	9a90aa67e6	common: resuscitate TestStepCreateFloppy_missing()	6 years ago
Lars Lehtonen	8e2885e626	common: resuscitate TestStepCreateFloppy_notfound()	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
Megan Marsh	a783a09ea3	Merge pull request #8269 from paulmey/paulmey/generalize-chroot ... [amazon/chroot] Move common/generic chroot builder steps to common directory	6 years ago
Paul Meyer	3be7d81ce2	Move common steps together	6 years ago
Paul Meyer	e6dfe301ac	Move CommandWrapper & ShellCommand to common	6 years ago
Lars Lehtonen	731e0332cf	common: collect dropped error	6 years ago
Megan Marsh	d15f43e4eb	Merge pull request #8232 from hashicorp/artifact-tree ... HCL2 configs - part one	6 years ago
Megan Marsh	e5becfc854	Merge pull request #8248 from marcinbojko/master ... hyper-v fix for compacting only .vhd|.vhdx files	6 years ago
Marcin Bojko	1b4ffe546f	hyper-v fix for compacting vhdx/vhd only	6 years ago