7946 Commits (ba7679b6791a306a73062ddd5a00359452e152fc)

Author	SHA1	Message	Date
Alex Harvey	cbeedfca35	provisioner/puppet: fix bug when connection type was not set in config (#23057) ... Before this, the Terraform Puppet provisioner would error out in a confusing way if the type attribute in a connection block was not given. Apparently an omitted type leads to type having a value "" which must be then assumed to mean "ssh". Fixes #23004	6 years ago
Pam Selle	b40385772e	Merge pull request #22705 from kmott/habitat-provisioner-updates ... Habitat provisioner updates	6 years ago
Kyle Mott	30895a6cf5	Merge cleanup, remove `license` parameter in favor of bool `accept_license`, adjust how license acceptance is done, update hab provisioner doc.	6 years ago
Kyle Mott	e3d1876f44	Merge upstream	7 years ago
James Bardin	a3c374b227	don't lose track of private data in the ACC tests ... If the resource schema version is > 0, the metadata containing the timeout values was overwritten during acceptance tests.	7 years ago
Pam Selle	80dec63917	Merge pull request #22691 from pselle/customizediff ... Evaluate ignore changes before sending plan to CustomizeDiff	7 years ago
Mudassar Shafique	50a6c69ab5	Habitat License Acceptance changes. These changes add a new argument that allows users to accept Habitat End User License. ... It also containts doc updates for Habitat and Chef provisioner	7 years ago
Pam Selle	2e5a8c0f6e	Update when ignore_changes are evaluated, to impact customizediff	7 years ago
Kyle Mott	997761ef41	Remove commented out code.	7 years ago
Kyle Mott	164332bfa1	Update Linux Habitat Provisioner to support new licensing, options cleanup.	7 years ago
James Bardin	a94f5ee132	prevent panics when encountering nil diffs ... While we can't change the behavior of helper/schema at this point, we can protect against panics in the case of unexpected nils in the instance diff.	7 years ago
Alex Pilon	c993fc3c1b	better error handling for type assertions	7 years ago
Alex Pilon	4bf43efcfd	move hcl2shim package to configs	7 years ago
Alex Pilon	d1448fc319	restore test ... switch provisioner validation to cast to []interface{} first	7 years ago
Alex Pilon	83aa07f907	prune NewResourceConfig and update tests	7 years ago
Pam Selle	291cf8c5a4	Revert "Habitat License Acceptance changes. "	7 years ago
Pam Selle	9020636f93	Merge pull request #21345 from mudash/master ... Habitat License Acceptance changes.	7 years ago
James Bardin	682286e184	test for Required win MinItems > 1	7 years ago
Alex Pilon	7f8f198719	remove UnknownVariabeValue from config and update references to shim	7 years ago
James Bardin	efd2a6cef6	dynamic set block test	7 years ago
James Bardin	a0338df4d4	update ignore_changes to use cty.Path.Equals ... Remove reflect.DeepEqual from path comparisons to get reliable results. The equality issues were only noticed going the grpc interface, so add a corresponding test to the test provider.	7 years ago
Radek Simko	5b9f2fafc8	Standardise directory name for test data	7 years ago
James Bardin	0d2363a058	add tests to preserve existing Set nil behavior ... While it may not be intuitive, providers expect that setting a `nil` value will appear as an empty string in state.	7 years ago
James Bardin	75602df5ef	Revert "Remove removed attribute from applied state" ... This reverts commit 2e2a363052.	7 years ago
James Bardin	0d9f84414a	Merge pull request #21879 from hashicorp/jbardin/sdk-panics ... prevent sdk panics in 2 specific cases	7 years ago
James Bardin	fe11724678	test for panic when readin empty map	7 years ago
James Bardin	f362198467	Merge pull request #21806 from hashicorp/jbardin/lost-empty-strings ... prevent an empty string from being lost	7 years ago
James Bardin	c4874aa5b3	add more timeout provider tests	7 years ago
James Bardin	4dcda04d96	prevent an empty string from being lost ... The helper/schema diff process loses empty strings, causing them to show up as unset (null) during apply. Besides failing to show as set by GetOk, the absence of the value also triggers the schema to insert a default value again during apply. It would also be be preferable if the defaults weren't re-evaluated again during ApplyResourceChange, but that would require a more invasive patch to the field readers, and ensuring the empty string is stored in the plan should block the default.	7 years ago
James Bardin	8ae31aa2db	normalize empty blocks during import ... Like Upgrade, Import is another case where we don't have the context of the configuration and need to ensure missing blocks are normalized.	7 years ago
James Bardin	2e2a363052	Remove removed attribute from applied state ... When a Diff contains a NewRemoved attribute (which would have been null in the planned state), the final value is often the "zero" value string for the type, which the provider itself still applies to the state. Rather than risking a change of behavior in helper/schema by fixing the inconsistency, we'll remove the NewRemoved attributes after apply to prevent further issues resulting from the change in planned value.	7 years ago
Tim Sharpe	615110e13e	provisioner: new Puppet provisioner (#18851) ... * Basic Puppet provisioner * (fixup) fix snake_case use in Bolt * (fixup) Remove unused ValidateFunc * (fixup) Check bolt result status * (lint) go fmt * Requested changes * Remove PE autodetection * Apply suggestions from @svanharmelen Co-Authored-By: rodjek <tim@sharpe.id.au> * Tag all JSON fields in bolt output * Defer comm.Disconnect() as suggested * Make bolt timeout configurable * Update builtin/provisioners/puppet/resource_provisioner.go Co-Authored-By: rodjek <tim@sharpe.id.au> * Make extension_requests and custom_attributes configurable	7 years ago
James Bardin	d33c5163a7	Merge pull request #21555 from hashicorp/jbardin/re-validate ... Allow providers to re-validate the final resource config	7 years ago
James Bardin	a056b84cdd	add delete timeout test ... The timeout value is still not being persisted. While it doesn't fix this issue, make sure to always return Private during Plan.	7 years ago
Mudassar Shafique	258319e6b0	Adding a comment to explain a testcase	7 years ago
James Bardin	c658fe173a	test a dynamic block with MinItems in the schema ... A dynamic block is always going to be a single value during validation, but should not fail when min-items > 1 is specified.	7 years ago
James Bardin	6055cb632e	filter unknowns from simple lists and maps in sdk ... While there was already a check in the sdk to filter unknowns from validation, it missed the case where those were in simple lists and maps.	7 years ago
Mudassar Shafique	e78658de4c	Habitat License Acceptance changes. These changes add a new argument that allows users to accept Habitat End User License.	7 years ago
James Bardin	c391f3a1a3	failing test for cty conversion bug	7 years ago
James Bardin	b9e5745b3c	remove SkipCoreTypeCheck and LegacySchema ... Update the test provider to match the sdk.	7 years ago
James Bardin	6bc36d3321	validate integers when using protoV5 ... The new type system only has a Number type, but helper schema differentiates between Int and Float values. Verify that a new config value is an integer during Validate, because the existing WeakDecode validation will decode a float value into an integer while the config FieldReader will attempt to parse the float exactly. Since we're limiting this to protoV5, we can be certain that any valid config value will be converted to an `int` type by the shims. The only case where an integral float value will appear is if the integer is out of range for the systems `int` type, but we also need to prevent that anyway since it would fail to read in the same manner.	7 years ago
James Bardin	7075bc9a4d	restrict the ComputedKeys usage to containers ... Computed primitive values must see the UnknownConfigValue or they are assumed to be unchanged. Restrict the usage of the protov5 ComputedKeys to containers.	7 years ago
Martin Atkins	083af21d30	providers/terraform: Explicit validate step ... We were previously catching some errors at read time, but some type errors were panicking because the cty.DynamicPseudoType arguments have no automatic pre-type-checking done but this code was assuming they would be objects. Here we add an explicit validation step that includes both the backend validation we were previously doing during read and some additional type checking to ensure the two dynamic arguments are suitably-typed. Having the separate validation step means that these problems can be detected by "terraform validate", rather than only in "terraform plan" or "terraform apply".	7 years ago
James Bardin	f9dfa98533	test to make sure a data source is planned ... The data source here needs to be re-read during apply, because its config is changing.	7 years ago
Martin Atkins	332010fd56	plans/objchange: Fix handling of dynamic block placeholders ... If a dynamic block (in the HCL dynamic block extension sense) has an unknown value for its for_each argument, it gets expanded to a single placeholder block with all of its attributes set to a unknown values. We can use this as part of a heuristic to relax our object compatibility checks for situations where the plan included an object that appears to be (but isn't necessarily) such a placeholder, allowing for the fact that the one placeholder block could be replaced with zero or more real blocks once the for_each value is known. Previously our heuristic was too strict: it would match only if the only block present was a dynamic placeholder. In practice, users may mix dynamic blocks with static blocks of the same type, so we need to be more liberal to avoid generating incorrect incompatibility errors in such cases.	7 years ago
James Bardin	0f883b118a	test that computed maps are applied correctly ... Verify that a computed map which is not correctly marked as unknown in the plan is still applied correctly.	7 years ago
James Bardin	67395306e1	delete unknown values from apply config altogether ... removeConfigUnknowns need to remove the value completely from the config map. Removing this value allows GetOk and GetOkExists to indicate if the value was set in the config in the case of an Optional+Computed attribute.	7 years ago
Martin Atkins	861a2ebf26	helper/schema: Use a more targeted shim for nested set diff applying ... We previously attempted to make the special diff apply behavior for nested sets of objects work with attribute mode by totally discarding attribute mode for all shims. In practice, that is too broad a solution: there are lots of other shimming behaviors that we _don't_ want when attribute mode is enabled. In particular, we need to make sure that the difference between null and empty can be seen in configuration. As a compromise then, we will give all of the shims access to the real ConfigMode and then do a more specialized fixup within the diff-apply logic: we'll construct a synthetic nested block schema and then use that to run our existing logic to deal with nested sets of objects, while using the previous behavior in all other cases. In effect, this means that the special new behavior only applies when the provider uses the opt-in ConfigMode setting on a particular attribute, and thus this change has much less risk of causing broad, unintended regressions elsewhere.	7 years ago
Martin Atkins	ff2de9c818	core: Keep old value on error even for delete ... When an operation fails, providers may return a null new value rather than returning a partial state. In that case, we'd prefer to keep the old value so that we stand the best chance of being able to retry on a subsequent run. Previously we were making an exception for the delete action, allowing the result of that to be null even when an error is returned. In practice that was a bad idea because it would cause Terraform to lose track of the object even though it might not actually have been deleted. Now we'll retain the old object even in the delete case. Providers can still return partial new objects if they were able to partially complete a delete operation, in which case we'll discard what we had before, but if the result is null with errors then we'll assume the delete failed entirely and so just keep the old state as-is, giving us the opportunity to refresh it on the next run to see if anything actually happened after all. (This also includes a new resource in the test provider which isn't used by the patch but was useful for some manual UX testing here, so I thought I'd include it in case it's similarly useful in future, given how simple its implementation is.)	7 years ago
Martin Atkins	bd1a215580	helper/resource: Ignore Removed attributes for ImportStateVerify ... Due to the lossiness of our legacy models for diff and state, shimming a diff and then creating a state from it produces a different result than shimming a state directly. That means that ImportStateVerify no longer works as expected if there are any Computed attributes in the schema where d.Set isn't called during Read. Fixing that for every case would require some risky changes to the shim behavior, so we're instead going to ask provider developers to address it by adding `d.Set` calls where needed, since that is the contract for "Computed" anyway -- a default value should be produced during Create, and thus by extension during Import. However, since a common situation where this occurs is attributes marked as "Removed", since all of the code that deals with them has generally been deleted, we'll avoid problems in that case here by treating Removed attributes as ignored for the purposes of ImportStateVerify. This required exporting some functionality that was formerly unexported in helper/schema, but it's a relatively harmless schema introspection function so shouldn't be a big deal to export it.	7 years ago