453 Commits (b709d27cf770cd40928fb91ec0feec87a486316c)

Author	SHA1	Message	Date
Mitchell Hashimoto	475d8750bb	command/push: make test more resilient	10 years ago
Radek Simko	8494cad8c4	Output 'destroy complete' when it's destroy (was: apply) (#8453)	10 years ago
Mitchell Hashimoto	2f8baa4580	command/push: fix tests to be vcs=false GH-8478	10 years ago
Mitchell Hashimoto	9a8209cfbd	command/push: create the proper parent directory entries in tar	10 years ago
Mitchell Hashimoto	928fdff33e	command/push: only add module directory to tar if it exists	10 years ago
Mitchell Hashimoto	92b15de080	command/push: remove the old test fixture for the new one	10 years ago
Mitchell Hashimoto	a650455ed6	command/push: only explicitly include the state ... modify the module include flag to flag whether we explicitly include or exclude modules	10 years ago
Mitchell Hashimoto	3aecc52bf3	command/push: "Extra" value must be absolute	10 years ago
Mitchell Hashimoto	f0de3c3e91	command/push: removed all the git stuff, turns out it doesn't matter	10 years ago
Mitchell Hashimoto	e8267f4907	command/push: failing test for pushing with no modules	10 years ago
Mitchell Hashimoto	70cc108614	Revert "command/push: test that -upload-modules=false works" ... This reverts commit edda576452.	10 years ago
Mitchell Hashimoto	edda576452	command/push: test that -upload-modules=false works	10 years ago
Mitchell Hashimoto	2ac142abc3	command: use bufio.ReadString instead of scanning to get spaces ... [GH-2628]	10 years ago
Mitchell Hashimoto	d3792e4aef	command: correct outdated comment	10 years ago
Mitchell Hashimoto	43cfd3d1c9	command: fix regressions for state mv with count resource	10 years ago
Mitchell Hashimoto	0d1ea84d39	command: test for moving resource with count [GH-7797]	10 years ago
Mitchell Hashimoto	3b3f92cd9b	terraform: fix some test failures on state add with multiple modules	10 years ago
Mitchell Hashimoto	a44c8b8760	terraform: state mv tests	10 years ago
James Nugent	f933b2cf16	Merge pull request #8200 from hashicorp/fix-state-rm ... core: Add `terraform state rm` command	10 years ago
Paul Stack	4d46812bab	Update state_rm_test.go ... Removing sort as it was imported and not used	10 years ago
Mitchell Hashimoto	3fdc08a9eb	core: Add `terraform state rm` command and docs ... This commit adds the `state rm` command for removing an address from state. It is the result of a rebase from pull-request #5953 which was lost at some point during the Terraform 0.7 feature branch merges.	10 years ago
Paul Hinze	84bbbfbe18	command: Fixup comment typo ... Hat tip to @Ninir :)	10 years ago
Paul Hinze	50df583ffd	Merge pull request #8120 from hashicorp/jbardin/null-state-module ... Fix panic from a null resources module in a state file	10 years ago
James Bardin	1be5a650fc	Add test for showing an empty state ... Make sure we don't panic if there's no state to print	10 years ago
James Bardin	191d7c1007	Fix panic when showing empty state ... Fixes a nil dereference when there's no state to print. Fix some slice declaration to use the recommended style when allocations don't matter.	10 years ago
James Bardin	cdb80f68a8	Ensure better state normalization ... Fix checksum issue with remote state If we read a state file with "null" objects in a module and they become initialized to an empty map the state file may be written out with empty objects rather than "null", changing the checksum. If we can detect this, increment the serial number to prevent a conflict in atlas. Our fakeAtlas test server now needs to decode the state directly rather than using the ReadState function, so as to be able to read the state unaltered. The terraform.State data structures have initialization spread out throughout the package. More thoroughly initialize State during ReadState, and add a call to init() during WriteState as another normalization safeguard. Expose State.init through an exported Init() method, so that a new State can be completely realized outside of the terraform package. Additionally, the internal init now completely walks all internal state structures ensuring that all maps and slices are initialized. While it was mentioned before that the `init()` methods are problematic with too many call sites, expanding this out better exposes the entry points that will need to be refactored later for improved concurrency handling. The State structures had a mix of `omitempty` fields. Remove omitempty for all maps and slices as part of this normalization process. Make Lineage mandatory, which is now explicitly set in some tests.	10 years ago
James Nugent	e181d753fe	build: Fix ordering of plugin list	10 years ago
Brad Sickles	70cadcf31d	Implement archive provider and "archive_file" resource. (#7322)	10 years ago
James Bardin	1322aaf473	Merge pull request #7989 from hashicorp/jbardin/tf_push ... Override atlas variables even if they aren't local	10 years ago
James Bardin	67bd4f29e0	Override atlas variables even if they aren't local ... Some Atlas usage patterns expect to be able to override a variable set in Atlas, even if it's not seen in the local context. This allows overwriting a variable that is returned from atlas, and sends it back. Also use a unique sential value in the context where we have variables from atlas. This way atals variables aren't combined with the local variables, and we don't do something like inadvertantly change the type, double encode/escape, etc.	10 years ago
James Bardin	d5fbb5f5c0	Modify the tfvars test to also use a cli var ... Modify the test to demonstrate where cli vars were being lost because they weren't interpreted as strings.	10 years ago
James Bardin	9d0faa2cae	Strip off extra \n in hcl encoded variables ... They don't change the value, but they do currently end up in the UI	10 years ago
James Bardin	bf83b435e1	numeric variables aren't always interpreted as str ... If we have a number value in our config variables, format it as a string, and send it with the HCL=true flag just in case. Also use %g for for float encoding, as the output is a generally a little friendlier.	10 years ago
James Bardin	1af7ee87a2	Silence log output when not verbose ... Set the default log package output to iotuil.Discard during tests if the `-v` flag isn't set. If we are verbose, then apply the filter according to the TF_LOG env variable.	10 years ago
James Bardin	2747d964cb	Merge pull request #7877 from hashicorp/jbardin/races ... core: Fix race conditions, mostly around diffs	10 years ago
James Nugent	9034196baf	Merge pull request #7875 from hashicorp/b-outputs-by-module ... core: Fix -module for terraform output command	10 years ago
James Nugent	0e4e94a86f	core: Fix -module for terraform output command ... The behaviour whereby outputs for a particular nested module can be output was broken by the changes for lists and maps. This commit restores the previous behaviour by passing the module path into the outputsAsString function. We also add a new test of this since the code path for indivdual output vs all outputs for a module has diverged.	10 years ago
James Bardin	5802f76eaa	Make all terraform package tests pass under -race ... This isn't a pretty refactor, but fixes the race issues in this package for now. Fix race on RawConfig.Config() fix command package races	10 years ago
James Bardin	0c714592f0	Fix variable handling on subsequent pushes ... The handling of remote variables was completely disabled for push. We still need to fetch variables from atlas for push, because if the variable is only set remotely the Input walk will still prompt the user for a value. We add the missing remote variables to the context to disable input. We now only handle remote variables as atlas.TFVar and explicitly pass around that type rather than an `interface{}`. Shorten the text fixture slightly to make the output a little more readable on failures.	10 years ago
James Bardin	b4b70193d2	whitespace fixes	10 years ago
James Bardin	8038e60a20	Add a function to quote HCL strings ... The strings we have in the variables may contain escaped double-quotes, which have already been parsed and had the `\`s removed. We need to re-escape these, but only if we are in the outer string and not inside an interpolation.	10 years ago
James Bardin	de87267697	Add tf_vars to the variables sent in push ... Add tf_vars to the data structures sent in terraform push. This takes any value of type []interface{} or map[string]interface{} and marshals it as a string representation of the equivalent HCL. This prevents ambiguity in atlas between a string that looks like a json structure, and an actual json structure. For the time being we will need a way to serialize data as HCL, so the command package has an internal encodeHCL function to do so. We can remove this if we get complete package for marshaling HCL.	10 years ago
James Nugent	681d94ae20	core: Allow lists and maps as variable overrides ... Terraform 0.7 introduces lists and maps as first-class values for variables, in addition to string values which were previously available. However, there was previously no way to override the default value of a list or map, and the functionality for overriding specific map keys was broken. Using the environment variable method for setting variable values, there was previously no way to give a variable a value of a list or map. These now support HCL for individual values - specifying: TF_VAR_test='["Hello", "World"]' will set the variable `test` to a two-element list containing "Hello" and "World". Specifying TF_VAR_test_map='{"Hello = "World", "Foo" = "bar"}' will set the variable `test_map` to a two-element map with keys "Hello" and "Foo", and values "World" and "bar" respectively. The same logic is applied to `-var` flags, and the file parsed by `-var-files` ("autoVariables"). Note that care must be taken to not run into shell expansion for `-var-` flags and environment variables. We also merge map keys where appropriate. The override syntax has changed (to be noted in CHANGELOG as a breaking change), so several tests needed their syntax updating from the old `amis.us-east-1 = "newValue"` style to `amis = "{ "us-east-1" = "newValue"}"` style as defined in TF-002. In order to continue supporting the `-var "foo=bar"` type of variable flag (which is not valid HCL), a special case error is checked after HCL parsing fails, and the old code path runs instead.	10 years ago
James Bardin	2712328d5a	Merge pull request #7756 from hashicorp/jbardin/tf-atlas-version ... core: send version info to atlas	10 years ago
James Bardin	885935962c	Add a terraform version header to all atlas calls ... Using the DefaultHeader added to the atlas.Client	10 years ago
David Glasser	50959a654c	command: Remove second DefaultDataDirectory const (#7666)	10 years ago
James Nugent	3735140286	core: Don't set variables for Atlas until lib is updated	10 years ago
James Nugent	5d18f41f04	core: Convert context vars to map[string]interface{} ... This is the first step in allowing overrides of map and list variables. We convert Context.variables to map[string]interface{} from map[string]string and fix up all the call sites.	10 years ago
Raphael Randschau	9081cabd6e	Add scaleway provider (#7331) ... * Add scaleway provider this PR allows the entire scaleway stack to be managed with terraform example usage looks like this: ``` provider "scaleway" { api_key = "snap" organization = "snip" } resource "scaleway_ip" "base" { server = "${scaleway_server.base.id}" } resource "scaleway_server" "base" { name = "test" # ubuntu 14.04 image = "aecaed73-51a5-4439-a127-6d8229847145" type = "C2S" } resource "scaleway_volume" "test" { name = "test" size_in_gb = 20 type = "l_ssd" } resource "scaleway_volume_attachment" "test" { server = "${scaleway_server.base.id}" volume = "${scaleway_volume.test.id}" } resource "scaleway_security_group" "base" { name = "public" description = "public gateway" } resource "scaleway_security_group_rule" "http-ingress" { security_group = "${scaleway_security_group.base.id}" action = "accept" direction = "inbound" ip_range = "0.0.0.0/0" protocol = "TCP" port = 80 } resource "scaleway_security_group_rule" "http-egress" { security_group = "${scaleway_security_group.base.id}" action = "accept" direction = "outbound" ip_range = "0.0.0.0/0" protocol = "TCP" port = 80 } ``` Note that volume attachments require the server to be stopped, which can lead to downtimes of you attach new volumes to already used servers * Update IP read to handle 404 gracefully * Read back resource on update * Ensure IP detachment works as expected Sadly this is not part of the official scaleway api just yet * Adjust detachIP helper based on feedback from @QuentinPerez in https://github.com/scaleway/scaleway-cli/pull/378 * Cleanup documentation * Rename api_key to access_key following @stack72 suggestion and rename the provider api_key for more clarity * Make tests less chatty by using custom logger	10 years ago
James Nugent	b4048dfc1d	core: Add -json flag to `terraform output` ... This commit removes the ability to index into complex output types using `terraform output a_list 1` (for example), and adds a `-json` flag to the `terraform output` command, such that the output can be piped through a post-processor such as jq or json. This removes the need to allow arbitrary traversal of nested structures. It also adds tests of human readable ("normal") output with nested lists and maps, and of the new JSON output.	10 years ago