80 Commits (828440d85289d515dde49d55bda68287fbae9f21)

Author	SHA1	Message	Date
Megan Marsh	24dd7e9dbb	Merge pull request #9257 from hashicorp/azr-hcl2-use-source-type-and-name ... HCL2: use source type and name as Name of a CoreBuild	6 years ago
Adrien Delorme	1ddf2a7d05	Update types.packer_config_test.go	6 years ago
Adrien Delorme	d431db0a38	HCL2: use source type and name as Name of a CoreBuild	6 years ago
Megan Marsh	ee5635722b	enable force, debug, and on-error for hcl2 builds	6 years ago
Adrien Delorme	1f3e85185e	Update types.packer_config.go	6 years ago
Adrien Delorme	0062160581	Update common_test.go	6 years ago
Adrien Delorme	42a05e1e80	more refactoring	6 years ago
Adrien Delorme	526d040534	document Parser.Parse better	6 years ago
Adrien Delorme	5c2b8da63b	get builds from PackerConfig instead of parser ... to allow just reading the config and to not start anything. This will allow to later on run `validate --syntax-only`. Note that none of the builder/provisioner/post-processor config will be read but simply ignored. HCL2 still needs the body to be properly formatted and it should detect most syntax errors.	6 years ago
Adrien Delorme	2df21496b3	hcl2template: let PackerConfig actually start a builder/provisioner/post-processor	6 years ago
Adrien Delorme	a3343c1848	Delete zz_retrocompat.go ... it's dead/unused code I previously though I'd need it	6 years ago
Tom Dyas	c0a6623ea2	teach HCL mode builds to honor -only and -except options (#8947)	6 years ago
Adrien Delorme	73242e8de4	make sure keyvalue and namevalue filters are using the same naming as the service being used	6 years ago
Sylvia Moss	553b1fb9f8	Add RetriedProvisioner to allow retry provisioners (#9061)	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
Adrien Delorme	2af40c762b	HCL2: pass on builder type and name (#8956) ... * sets `packer_build_name` and `packer_builder_type` variables for builder provisioners and post-processors in HCL2 * allows to use the new `${source.type}` and `${source.name}` variables in HCL2 * fixes #8932 Note that the common.PackerConfig is used everywhere and was not set for HCL2, this had some implications: For #8923 you can see the issue here: dde74232f2/builder/lxd/config.go (L61-L63) More random examples of where this could cause an issue : 0785c2f6fc/provisioner/ansible-local/provisioner.go (L380-L381) b4efd13a4d/builder/amazon/ebs/builder.go (L232-L236) * [All references to PackerConfig.PackerBuildName](https://sourcegraph.com/github.com/hashicorp/packer@ff6a039d5bb45e34ff761d9c52e8b98972288447/-/blob/common/packer_config.go#L7:2&tab=references) * [All references to PackerConfig.PackerBuilderType](https://sourcegraph.com/github.com/hashicorp/packer@ff6a039d5bb45e34ff761d9c52e8b98972288447/-/blob/common/packer_config.go#L8:2&tab=references)	6 years ago
Jeff Escalante	9165be87da	packer.io prefix removal, html extension remove for in-code errors	6 years ago
Adrien Delorme	d068430abf	make sure locals are evaluated only once variables are + test this (#8918) ... fix #8898	6 years ago
Adrien Delorme	ad8dafa3bd	HCL: add tests and fixes around var-file and var args (#8914)	6 years ago
Adrien Delorme	88297c796d	Merge branch 'master' into hcl2_singular_blocks	6 years ago
Adrien Delorme	3beea4fb94	NameValues.CopyOn: don't instanciate a receiving map if there is nothing to copy over	6 years ago
Adrien Delorme	b5e8750f5f	Change key/value to name/value to remain consistent with Terraform	6 years ago
Adrien Delorme	c845436e32	KeyValues.CopyOn: make sure a receiving nil map is set too	6 years ago
Adrien Delorme	87d6b2433f	make generate	6 years ago
Adrien Delorme	6e6490d688	WIP	6 years ago
Adrien Delorme	af38430eb8	add kvfilter type that regroups very frequently used filters	6 years ago
Adrien Delorme	9e6d5da277	hcl2: add general kv store that could be used anywhere	6 years ago
Adrien Delorme	c9ec967205	better testing	6 years ago
Adrien Delorme	710ebdcef5	add more test for var files	6 years ago
Adrien Delorme	c8300b620a	allow to use hcl files as var files in HCL mode ... fix #8781	6 years ago
zac	79f1fab836	hcl2template/types: add spaces in error messages. (#8881)	6 years ago
Megan Marsh	e518a0a8b7	Merge branch 'master' into update_go-cty_regex	6 years ago
Adrien Delorme	0c45bd8b4f	import new replace and regex replace funcs from go-cty + documentation	6 years ago
Adrien Delorme	631f402d1e	Merge branch 'master' into fix_8730	6 years ago
Adrien Delorme	0ccff0d5b9	all variables must have a value. ... A variable's default value can be set to null to force user to set it.	6 years ago
Adrien Delorme	6d8cce501e	tweak validation & add tests	6 years ago
Wilken Rivera	9403b48a52	hcl2template/types.variables: Update logic for parsing literal value variables (#8834) ... * hcl2template/types.variables: Update logic for parsing literal value variables In running tests via the CLI it was determined that when using the variable block with no explicit type assigned the type of the default value was not being set within the map. This change updates the `decodeConfig` method so that a type is always set for any defined variable if not specified. The second change is to properly handle the evaluation of basic variable types (e.g String, Number, Bool). Previously variables defined on the CLI or via PKR_VAR required some additional quoting to for proper evaluation. This change fixes that issue so that it works like it does in Terraform :) Build results before the change ``` ⇶ PKR_VAR_example='["one","two"]' ~/bin/packer build -var 'foo=home' . Error: Variables not allowed on <value for var.foo from arguments> line 1: (source code not available) Variables may not be used here. ==> Builds finished but no artifacts were created. ``` Build results after the change ``` ⇶ PKR_VAR_example='["one","two"]' ~/bin/packer build -var 'foo="home"' . null: output will be in this color. ==> null: Running local shell script: /tmp/packer-shell885249462 null: two null: home Build 'null' finished. ==> Builds finished. The artifacts of successful builds are: --> null: Did not export anything. This is the null builder ⇶ ~/bin/packer build -var 'foo=home' -var 'example=["one","another variable"]' . null: output will be in this color. ==> null: Running local shell script: /tmp/packer-shell123467506 null: another variable null: home Build 'null' finished. ==> Builds finished. The artifacts of successful builds are: --> null: Did not export anything. This is the null builder ``` * tests/hcl2template/types.variables: Update test to use Bool Turns out a string value won't actually complain if it's given a non string looking value. It will just covert the value to a string literal so using a type Bool which should fail if given anything that is not true or false. * tests/hcl2template/types.variables: Update unit tests During testing it was found that by default the variable stanza were defaulting to a cty.NilType, and not the Type of it's default value. This change sets the default type of the defined variable to ensure variable evaluation behaviors correctly. * Add a simple cty.Bool test case * tests/hcl2template/types.variables: Enable quoted_string test case * Update hcl2template/types.variables.go space Co-authored-by: Adrien Delorme <azr@users.noreply.github.com>	6 years ago
Adrien Delorme	e9d3826219	variables: use Range instead of Block ... because a variable can be defined in the Locals block	6 years ago
Adrien Delorme	9b649594c6	update tests	6 years ago
Adrien Delorme	8482c6c2e6	evaluateLocalVariable: also pass the variable name	6 years ago
Adrien Delorme	bd3112fa7c	parseLocalVariables: simplify loop	6 years ago
Adrien Delorme	23b940dbd5	fix this !	6 years ago
Adrien Delorme	e851050774	don't force to set unused variables too	6 years ago
Adrien Delorme	382f209b07	Update types.packer_config.go	6 years ago
Adrien Delorme	59cc246942	Merge remote-tracking branch 'origin/master' into fix_8730	6 years ago
Adrien Delorme	50896d4ddf	WIP	6 years ago
Sylvia Moss	591b684f08	Fix HCL2 local variables decoding to allow local usage within another local in the same locals block (#8755)	6 years ago
Adrien Delorme	a8fcb2d91a	HCL2: add support for dynamic blocks, document for loops and splat expressions (#8720)	6 years ago
Adrien Delorme	52e5b7051e	Update mock.go	6 years ago
Adrien Delorme	3c213e6eaf	continue and realise and unused undefaulted value triggers an error	6 years ago