84 Commits (v1.6.3)

Author	SHA1	Message	Date
zhsj	e2cbe0de48	post-processor/vagrant-cloud: missing vagrant_cloud_url in test (#9572)	6 years ago
Adrien Delorme	40947c2bf6	HCL2: generate hcl tags with go-cty tags too (#9306) ... This will allow to generate the config files: ```go package main import ( "fmt" "github.com/hashicorp/hcl/v2/gohcl" "github.com/hashicorp/hcl/v2/hclwrite" "github.com/hashicorp/packer/builder/alicloud/ecs" ) func main() { name := "name" app := ecs.FlatConfig{ AlicloudImageName: &name, ECSSystemDiskMapping: &ecs.FlatAlicloudDiskDevice{ DiskName: &name, }, } f := hclwrite.NewEmptyFile() block := gohcl.EncodeAsBlock(&app, `source "something" "something"`) f.Body().AppendBlock(block) fmt.Printf("%s", f.Bytes()) } ``` Will output: ``` source "something" "something" { packer_user_variables = null packer_sensitive_variables = null image_name = "name" image_share_account = null image_unshare_account = null image_copy_regions = null image_copy_names = null tags = null tag = null system_disk_mapping = { disk_category = null, disk_delete_with_instance = null, disk_descri ption = null, disk_device = null, disk_encrypted = null, disk_name = "name", disk_size = null, disk_ snapshot_id = null } image_disk_mappings = null ssh_remote_tunnels = null ssh_local_tunnels = null ssh_public_key = null ssh_private_key = null } ``` This is a good first step for #9015 and #9282 fix #9304	6 years ago
Megan Marsh	b982d987a7	Add Artifice postprocessor ID to valid artifacts for postprocessors that perform artifact type validation	6 years ago
Adrien Delorme	840e67f775	HCL2: when we see a map generate an attribute spec instead of a block spec (#9035) ... * mapstructure-to-hcl2: when we see a map generate an attribute spec and not a block spec this will alow to do tags = { key = "value" } instead of tags { key = "value" } This will also enable using variables directly for those tags * generate code * update tests	6 years ago
Sylvia Moss	dc31bad539	Sharing info with post-processors via artifact (#8632)	6 years ago
nywilken	5a54c99e98	post-processor/vagrant-cloud: Update error handling for VagrantCloud API responses ... This change updates the underlying type of VagrantCloudErrors.Errors to match the expected errors response type and adds a simple test to verify the parsing of various error responses. Tests before change ``` --- FAIL: TestVagranCloudErrors (0.00s) client_test.go:23: failed to decode error response: json: cannot unmarshal array into Go struct field VagrantCloudErrors.errors of type map[string][]string client_test.go:23: failed to decode error response: json: cannot unmarshal array into Go struct field VagrantCloudErrors.errors of type map[string][]string client_test.go:26: failed to get expected response; expected "", got "error1. error2" client_test.go:23: failed to decode error response: json: cannot unmarshal array into Go struct field VagrantCloudErrors.errors of type map[string][]string client_test.go:26: failed to get expected response; expected "", got "message Bad credentials" ``` Test after change ``` --- PASS: TestVagranCloudErrors (0.00s) ```	6 years ago
Megan Marsh	b23425f637	use pointer to config rather than config	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
Lars Lehtonen	2e7ba19d8c	post-processor/vagrant-cloud: fix dropped test errors	6 years ago
Adrien Delorme	078ba7c8c3	commit old code generation tool ... for history	6 years ago
Lars Lehtonen	21f2e67462	post-processor/vagrant-cloud: fix dropped errors	7 years ago
Brian Fox	331babc2ce	allow blank access_token (no auth) for private vagrant box hosting	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
DanHam	57308b0126	Should return an error with an invalid BuilderId	7 years ago
Adrien Delorme	d72040f4fa	move retry code into the common/retry pkg and make retry context aware	7 years ago
Adrien Delorme	a81abd297b	Merge remote-tracking branch 'origin/master' into context_provisioner	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	e65115a7a0	contextualize post-processor	7 years ago
Megan Marsh	12fc1fa751	default_keep_input_artifact	7 years ago
Vadym Haidamaka	4855bc79c9	Add vagrant-cloud post-processor support for the vagrant builder	7 years ago
Davi Vidal	b0e3128125	Improves test message	7 years ago
Davi Vidal	1a94ad14bd	Adds test for insecure_skip_tls_verify	7 years ago
Davi Vidal	a2fd287e56	Ignores SSL verification when on premise vagrant cloud	7 years ago
Megan Marsh	c63b54a1e7	move http with proxy call into a helper function	7 years ago
Adrien Delorme	7b10079507	fail if the request could not be created	7 years ago
Adrien Delorme	aa2ebec8fd	remove duplicate error message from	7 years ago
Adrien Delorme	ff4b6d4442	pass vagrant cloud auth token as http header ... * stoped using url to pass the auth token and put it in the headers * added newRequest method to VagrantCloudClient that sets json and auth headesr * made VagrantCloudClient method pointers to avoid copies	7 years ago
Adrien Delorme	415b886f5b	post-processor/vagrant-cloud: validate vagrant cloud auth token doing an auth request	7 years ago
Edouard BONLIEU	7e36cfcff1	Add Scaleway builder	8 years ago
Matthew Hooker	7a189a83a1	fix imports ... `find . -type f -name '*.go' -not -path "./vendor/*" -exec goimports -w {} \;`	8 years ago
Matthew Hooker	a831d522be	change run signatures ... Run now takes a context as well as a statebag. We'll assign the context to the blank identifier to prevent namespace collisions. We'll let the step authors opt-in to using the context. `find . -iname "step_*.go" -exec gsed -i'' 's/func \(.*\)Run(/func \1Run(_ context.Context, /' {} \;`	8 years ago
Matthew Hooker	366dc3da0a	move multistep imports to helper. ... gomvpkg -from "github.com/mitchellh/multistep" -to "github.com/hashicorp/packer/helper/multistep"	8 years ago
Matthew Hooker	ebe995c0ff	run goimports	8 years ago
Matthew Hooker	52558e4f75	check for nil body from upload response	8 years ago
Vijaya Bhaskar Reddy Kondreddi	e45223a867	Fix -on-error issue not working. ... When we supply -on-error flag for Packer with Hyper-v builder, it does not do anything.	9 years ago