569 Commits (412df59ea57d0793dd4f449ebde48db6bd471318)

Author	SHA1	Message	Date
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	39fd462b56	change all provision func signatures to use map[string]interface{}	6 years ago
Lars Lehtonen	20d375f705	post-processor/digitalocean-import: drop unused test functions	6 years ago
Lars Lehtonen	d7eee558a6	post-processor/docker-import: drop unused test functions (#8465)	6 years ago
Lars Lehtonen	4fe34a5dc3	post-processor/docker-push: drop unused test helper functions (#8456)	6 years ago
Lars Lehtonen	8146b39986	post-processor/docker-save: remove unused test functions (#8447)	6 years ago
Adrien Delorme	b4200c3590	go generate for the google compute export post processor after #8388	6 years ago
Taneli Leppä	e1cb0c5f7e	Added support for specifying Service Account in Google Compute exporter post-processor. (#8388)	6 years ago
henry-aj-jones	d317d7f3d1	post-processor/docker-tag: Allow to tag an artifact with multiple tags (#8392)	6 years ago
Lars Lehtonen	2e7ba19d8c	post-processor/vagrant-cloud: fix dropped test errors	6 years ago
Megan Marsh	fcb65ee422	Merge pull request #8307 from hashicorp/fix_8116 ... delete amazon import post-processor intermediary snapshots	6 years ago
Megan Marsh	e9e4c102e5	delete amazon import post-processor intermediary snapshots	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
mingsheng.su	4f140d0071	fix docs of ucloud-uhost	6 years ago
mingsheng.su	545575b879	optimize code about ufile config	6 years ago
mingsheng.su	37e898e115	optimize code style	6 years ago
mingsheng.su	1099777978	update bucket about no create	6 years ago
mingsheng.su	083d7896f4	update message of ucloud import	6 years ago
mingsheng.su	2c513fce64	update ucloud packer	6 years ago
mingsheng.su	61f9d3a9fe	add ucloud import	6 years ago
Lars Lehtonen	5edabfc849	post-processor/compress: fix dropped errors	6 years ago
Adrien Delorme	078ba7c8c3	commit old code generation tool ... for history	6 years ago
Adrien Delorme	771a6e55ef	pp.google.[import|export]: unexport Account field ... so it doesnt temper with hcl2 cfg generation	6 years ago
Adrien Delorme	af08e2a410	pp.alicloud-import: remove duplicate AlicloudImageName field ... ( already defined in packerecs.Config )	6 years ago
Andrew Starr-Bochicchio	ae1bb856a3	digitalocean-import: Fix panic when 'image_regions' not set (Fixes: #7843).	6 years ago
Lars Lehtonen	21f2e67462	post-processor/vagrant-cloud: fix dropped errors	7 years ago
Megan Marsh	3c14c50aba	Allow user to real oauth token from properly configured Vault instance	7 years ago
Megan Marsh	61b237b6f7	Merge pull request #8097 from onematchfox/vagrant-cloud ... post-proc(vagrant-cloud) allow blank access_token (no auth) for private vagrant box hosting	7 years ago
Megan Marsh	9e28b2f5cf	fix post-processor	7 years ago
Brian Fox	331babc2ce	allow blank access_token (no auth) for private vagrant box hosting	7 years ago
Robbert Müller	9f972375ed	Removed whitelist check for builders ... Fixes https://github.com/hashicorp/packer/issues/8060 Signed-off-by: Robbert Müller <spam.me@grols.ch>	7 years ago
DanHam	ca9b500e76	Fix test output: Correct copy/paste errors and make message clearer	7 years ago
DanHam	7770ade74c	Clearer error message for artifice users. OCD nits and logging	7 years ago
DanHam	e9ab2203ba	Should return provider correctly with artifacts from other builders or pp's	7 years ago
DanHam	e8336039d9	Should return provider correctly with good box and artifice pp	7 years ago
DanHam	aee400836f	Should return an error if the provider kv pair is not in the metadata file	7 years ago
DanHam	a7603f63c7	Should return an error if the metadata file has badly formatted JSON	7 years ago
DanHam	57137c6e33	Should return an error if the value of the provider key is empty	7 years ago
DanHam	d1327fe422	Should return an error if the metadata file is empty	7 years ago
DanHam	a9e22a6bb2	Should return the provider by parsing the json in the box metadata file	7 years ago
DanHam	0bf0e7c078	Should return an error when the metadata file is not in the box tar archive ... Split out box creation into new helper function	7 years ago
DanHam	063e4bd3e5	Should return an error if the box tar archive is empty	7 years ago
DanHam	9c6b355088	Should return an error if the box is a plain gzip file	7 years ago
DanHam	35d326de39	Add basic workings to function. Return an error if box file is empty	7 years ago
DanHam	6b5cf6dcb2	Should return an error when the box file is missing	7 years ago
DanHam	e8c586175e	Intention: Allow use of artifice pp with vagrant-cloud pp ... The Vagrant-Cloud and Vagrant provider (e.g. virtualbox, vmware_desktop etc.) must be determined differently depending on the builder or post-processor supplying the artifact. Adds a wrapper function that: * Uses the original method of determining the provider when the artifact is provided by either the Vagrant builder or Vagrant post-processor * Uses a new (currently empty) function when the artifact is provided via the Artifice post-processor	7 years ago
DanHam	3f4f429c3d	Should return an error when artifact file does not have .box extension	7 years ago