b787fe2798
Merge pull request #5012 from jrnt30/TF-5011-Backend-flag-normalization
...
Fixes #5011 - Backend downcased for init
2016-02-08 17:59:20 -05:00
1628f19bc1
Fixing some golint issues on the new validate command
2016-02-08 22:04:24 +00:00
db69a2959b
Added verify command
2016-02-08 12:48:14 +01:00
61240b4250
Fixes #5011 - Backend downcased for init
2016-02-05 06:26:12 -06:00
4a51e4fe91
core: write planfile even on empty plans
...
This makes the planfile workflow more consistent. If a plan yields a
noop, the apply of that planfile will noop.
Fixes #1783
2016-01-20 16:00:20 -06:00
e67fc0fe9b
command: Change module-depth default to -1
...
This means that terraform commands like `plan`, `apply`, `show`, and
`graph` will expand all modules by default.
While modules-as-black-boxes is still very true in the conceptual design
of modules, feedback on this behavior has consistently suggested that
users would prefer to see more verbose output by default.
The `-module-depth` flag and env var are retained to allow output to be
optionally limited / summarized by these commands.
2016-01-20 13:58:02 -06:00
6a972a7713
command/init: put remote state config at proper path [GH-2927]
2016-01-19 17:13:19 -08:00
a5e51f6dad
command/refresh: Fix import formats to prevent flapping
2015-11-27 15:10:42 +00:00
0e277a6714
core: test coverage around map key regression
...
tests to cover the HCL-level issue fixed in
https://github.com/hashicorp/hcl/pull/65
2015-11-24 16:00:02 -06:00
50b5e7c1a5
Validate context after input of vars on refresh
...
Fixes #4013 .
2015-11-23 10:33:40 +02:00
890e214c00
Add failing test replicating #4013
2015-11-23 10:32:46 +02:00
39e7f42490
Clear up `destroy --target` message
...
When destroying infrastructure with `--target`, print out which
infrastructure will be destroyed instead of saying `Terraform will
delete all your managed infrastructure`.
```
terraform destroy --target aws_instance.test2 --target aws_instance.test1
Do you really want to destroy?
Terraform will delete the following infrastructure:
aws_instance.test2
aws_instance.test1
There is no undo. Only 'yes' will be accepted to confirm
```
Omitting `--target` arguments will use the default input description.
```
$ terraform destroy
Do you really want to destroy?
Terraform will delete all your managed infrastructure.
There is no undo. Only 'yes' will be accepted to confirm.
```
2015-11-11 12:04:58 -05:00
a49b162dd1
Prompt for input variables before context validate
...
Also adds a regression test using Mock UI. Fixes #3767 .
2015-11-10 14:03:56 -05:00
f4c03ec2a6
Reflect new comment format in stringer.go
...
As of November 8th 2015, (4b07c5ce8a), the word "Code" is prepended to
the comments in Go source files generated by the stringer utility.
2015-11-09 11:38:51 -05:00
9428e9f1d1
command: fix failing TestPlan_noState test
...
Turns out that isolating statefiles in the Apply tests is an important
step to prevent leaking .tfstate in a common dir, which can affect other
tests.
2015-10-29 18:07:44 -05:00
715437537d
command: fix flaky parallelism tests
...
Thanks to @mitchellh for helping sort out concurrency stuff here.
2015-10-29 15:16:34 -05:00
61e890ecc9
Update list of backends in RemoteConfigCommand's Help() method
2015-10-25 16:46:02 -04:00
e4e9c13c20
Merge #3136 : Colorize the 'forces new resource' message
2015-10-22 08:44:37 -07:00
344e7c26b5
fix a bunch of tests from go-getter import
2015-10-15 13:48:58 -07:00
e4845f75cc
removed extra parentheses
2015-10-08 15:48:04 +03:00
374070d066
website: docs for parallelism setting
...
/cc @stack72 @knuckolls @mitchellh
2015-10-05 17:21:29 -05:00
e1a46904d6
command: pull parallelism default up to CLI layer
...
/cc @knuckolls @josephholsten
2015-10-05 15:06:08 -05:00
bf9c5c46d0
Tests for apply parallelism=1 and parallelism=2
2015-09-29 22:41:26 -07:00
fc60b2858c
Added -parallelism to refresh and plan cli UI
2015-09-29 22:41:26 -07:00
f59c71b35a
works for apply, no tests yet
2015-09-29 22:41:26 -07:00
57bea9f26c
Merge pull request #3223 from scalp42/typos
...
remove various typos
2015-09-14 07:48:59 +01:00
4d37704d53
command/get: fix typo
2015-09-13 14:40:36 -07:00
198e1a5186
remove various typos
2015-09-11 11:56:20 -07:00
7c56550633
In plan summary, be more honest about force-new changes.
...
Previously the plan summary output would consider -/+ diffs as changes
even though they actually destroy and create instances. This was
misleadning and inconsistent with the accounting that gets done for the
similar summary written out after "apply".
Instead we now count the -/+ diffs as both adds and removes, which should
mean that the counts output in the plan summary should match those in
the apply summary, as long as no errors occur during apply.
This fixes #3163 .
2015-09-04 08:07:16 -07:00
2e51915431
Colorize the 'forces new resource' message.
...
Sometimes in all the output from ```terraform plan```, it is difficult
to see the ```(forces new resource)``` message.
This patch adds a little bit of color.
2015-08-31 15:37:09 -07:00
5a12608dc3
tests: remove unnecessary symlink
2015-08-31 15:45:43 -05:00
9596271b36
Code formatted via gofmt
2015-08-14 12:02:32 +01:00
10f04c90a5
'terraform output' helptext spacing fixed
2015-08-14 12:01:58 +01:00
382c0ba23d
command/output: Update test since we now allow printing everything
2015-08-03 12:44:41 +01:00
85d1d15d81
output command supports printing all outputs
...
if no output name is specified all outputs are displayed
fixed formating and added missing help for -module parameter
2015-08-03 12:22:40 +01:00
7a60174dd1
Do not print errors via UiHook
2015-07-22 14:09:23 +02:00
4a8ef78d33
Fixes #2676 by prefixing all Windows commands
...
By prefixing them with `cmd /c` it will work with both `winner` and
`ssh` connection types.
This PR also reverts some bad stringer changes made in PR #2673
2015-07-10 12:56:27 +02:00
97fd4f5b7d
Tweaking the tests
2015-07-09 21:29:27 +02:00
e376f61d1d
core: Return correct number of planned updates
2015-07-03 13:22:10 +02:00
1f92dd5b40
command/push: update output
2015-06-29 13:58:54 -07:00
c5d3c585c6
command/push: update var name
2015-06-29 13:57:58 -07:00
9d9bcc2f6e
command/push: update flag to -overwrite, update docs
2015-06-29 13:53:05 -07:00
8ee3281858
command/push: UX
2015-06-29 13:41:07 -07:00
dc40f044f0
command/push: prefer Atlas over local, add -set flag
2015-06-29 12:24:13 -07:00
8ebdc1e786
terraform: orphans should call post-apply hook [GH-1938]
2015-06-25 20:11:29 -07:00
a76105b0f1
command: make error message clearer for invalid tfvars format [GH-1889]
2015-06-25 17:56:37 -07:00
96b53cc06c
command: ask for input even if tfvars is set [GH-2161]
2015-06-25 15:29:46 -07:00
b505d15e1e
Add resource sums to terraform plan
...
- closes #2355
2015-06-25 10:25:16 +01:00
3f2119c6e7
Merge pull request #2414 from TimeIncOSS/no-color-for-subcommands
...
Add -no-color option for subcommands
2015-06-23 15:43:37 -05:00
74fb179127
Merge pull request #2371 from hashicorp/b-fix-tf-show-with-remote-state
...
core: fix `terraform show` with remote state
2015-06-23 15:01:10 -05:00
b7d41d2eed
Add -no-color to help text
2015-06-22 13:14:01 +01:00
cdcd11419e
Add -no-color option for subcommands
2015-06-21 21:52:11 +01:00
cb5c2567b3
core: fix `terraform show` with remote state
...
fixes #1810
2015-06-16 12:26:04 -05:00
ad680b1832
core: show outputs after `terraform refresh`
...
closes #25
2015-06-15 10:41:42 -05:00
18924d24fe
command/apply: flatten multierrors
...
Before:
```
Error applying plan:
1 error(s) occurred:
* 1 error(s) occurred:
* 1 error(s) occurred:
* 1 error(s) occurred:
* Error creating ...
```
After:
```
Error applying plan:
1 error(s) occurred:
* Error creating ...
```
Muuuch better! :D
2015-06-07 21:32:00 -05:00
aa39dc71de
command/output: fix error msg typo
2015-06-03 09:24:20 -05:00
b05e36a6e3
Add module outputs
2015-05-27 16:46:12 +02:00
77a61157fb
command/push: var/var-file supported, note in help [GH-1950]
2015-05-13 20:18:40 -07:00
8ce3e1d5b4
command/push: local vars override remote ones
...
Otherwise once you push a variable it becomes very difficult to change.
2015-05-08 18:54:38 -05:00
bb4f7415ca
Rename "The The" so that they can play songs again
...
Other than the fact that "The the" doesn't really make any sense anywhere
that it's used in Terraform, they're a post-punk band from the UK.
Fixes "The The" so that they can get back to playing songs.
2015-05-06 22:53:11 +01:00
29d34cd5a4
command: allow module depth to be set via env var
...
Another convienence env var here with TF_MODULE_DEPTH.
Works like you'd expect it to!
2015-04-30 16:19:43 -05:00
d30d88e327
Merge pull request #1655 from hashicorp/f-build-graph-during-plan
...
core: validate on verbose graph to detect some cycles earlier
2015-04-30 16:08:33 -05:00
5f1ea7fccc
command: add env var equivalent to -input arg
...
Setting TF_INPUT has the equivalent effect of setting the flag.
I got sick of specifying this when iterating on TF config locally.
2015-04-30 09:59:14 -05:00
b08ba78341
adding new AskSecret to ColorizeUi
2015-04-29 13:07:13 -05:00
ce49dd6080
core: graph command gets -verbose and -draw-cycles
...
When you specify `-verbose` you'll get the whole graph of operations,
which gives a better idea of the operations terraform performs and in
what order.
The DOT graph is now generated with a small internal library instead of
simple string building. This allows us to ensure the graph generation is
as consistent as possible, among other benefits.
We set `newrank = true` in the graph, which I've found does just as good
a job organizing things visually as manually attempting to rank the nodes
based on depth.
This also fixes `-module-depth`, which was broken post-AST refector.
Modules are now expanded into subgraphs with labels and borders. We
have yet to regain the plan graphing functionality, so I removed that
from the docs for now.
Finally, if `-draw-cycles` is added, extra colored edges will be drawn
to indicate the path of any cycles detected in the graph.
A notable implementation change included here is that
{Reverse,}DepthFirstWalk has been made deterministic. (Before it was
dependent on `map` ordering.) This turned out to be unnecessary to gain
determinism in the final DOT-level implementation, but it seemed
a desirable enough of a property that I left it in.
2015-04-27 09:23:47 -05:00
d4b9362518
core: validate on verbose graph to detect some cycles earlier
...
Most CBD-related cycles include destroy nodes, and destroy nodes were
all being pruned from the graph before staring the Validate walk.
In practice this meant that we had scenarios that would error out with
graph cycles on Apply that _seemed_ fine during Plan.
This introduces a Verbose option to the GraphBuilder that tells it to
generate a "worst-case" graph. Validate sets this to true so that cycle
errors will always trigger at this step if they're going to happen.
(This Verbose option will be exposed as a CLI flag to `terraform graph`
in a second incoming PR.)
refs #1651
2015-04-23 11:07:13 -05:00
e91b5136bf
Merge pull request #1356 from koendc/b-remote-plugin-case
...
command/state: fix for state files containing an upper case remote type
2015-04-14 08:34:28 -07:00
a2cb77058c
command: fix push tests when running outside repo
...
VCS detection was on by default, and blows up when the tests are run in
a copy of the Terraform source that is not a git repository, like - say
- during a Homebrew formula install, just to pick a random example. :)
2015-04-09 18:46:25 -05:00
650a9f372b
command: implement proper module.Storage interface
2015-04-07 16:46:56 -07:00
000d28c489
command/push: read terraform.tfvars
2015-04-07 15:34:06 -07:00
1fac7b6488
state: cache should use State.Equal to check equality
...
reflect.DeepEqual was returning false, sometimes.
2015-04-07 15:03:04 -07:00
428fd98ba0
Handles upper case characters in the cached state file's remote type
...
If the cached state file contains a remote type field with upper case
characters, eg 'Consul', it was no longer possible to find the 'consul'
remote plugin.
2015-04-01 21:42:39 +02:00
9a091ffa78
command: plan supports detailed exit code
2015-04-01 09:38:19 -07:00
97acccd3ed
core: targeted operations
...
Add `-target=resource` flag to core operations, allowing users to
target specific resources in their infrastructure. When `-target` is
used, the operation will only apply to that resource and its
dependencies.
The calculated dependencies are different depending on whether we're
running a normal operation or a `terraform destroy`.
Generally, "dependencies" refers to ancestors: resources falling
_before_ the target in the graph, because their changes are required to
accurately act on the target.
For destroys, "dependencies" are descendents: those resources which fall
_after_ the target. These resources depend on our target, which is going
to be destroyed, so they should also be destroyed.
2015-03-31 14:49:38 -05:00
da7f307e56
command/remote-config: failing tests
2015-03-26 18:14:24 -07:00
4a7b554cf7
command/remote-config: do a pull with `terraform remote config`
2015-03-26 17:57:45 -07:00
6379a888fb
command/remote-{pull,push}: colorize and show success output
2015-03-26 17:40:39 -07:00
38b1a727bf
command/remote-config: lowercase the type so that Atlas works, for
...
example
2015-03-26 17:37:05 -07:00
7bfa5afd00
command/remote-config: show flag parse errors
...
/cc @sethvargo
2015-03-26 17:33:58 -07:00
35da19cc1f
command/remote-config: remove weird error case that shows no error
...
message
/cc @sethvargo
2015-03-26 17:31:50 -07:00
1adb3fbfa0
command: when setting up state, only write back if local is newer
2015-03-26 17:16:54 -07:00
83cb277583
command/output: don't panic if no root module in state [GH-1263]
2015-03-25 16:35:27 -07:00
431f5e6706
command/remote-config: syntax
2015-03-25 15:26:38 -07:00
2796f83950
command/remote-config: make the error message nicer for remote init
...
/cc @sethvargo - based on UX issues you ran into yesterday
2015-03-25 15:23:04 -07:00
280635d2b0
command/push: add -atlas-address
2015-03-24 17:45:19 -07:00
51614b6365
command/push: upload-modules
2015-03-24 17:42:40 -07:00
8d47f26bb7
command/push: address PR comments
2015-03-24 17:41:26 -07:00
0229852e48
command/push: change wording on error message
2015-03-24 17:39:37 -07:00
395dd04861
command/push: accept -vcs
2015-03-24 17:03:59 -07:00
86f92119e2
command/push: no version on GET should be okay
2015-03-24 13:59:13 -07:00
19c1771438
command/push: integrate atlas push client
2015-03-24 13:42:48 -07:00
1c0ffbd7bf
command/push: output the name/version of the pushed configuration
2015-03-24 13:30:23 -07:00
ee1ad49829
command/push: rename -module-lock to -module-upload
2015-03-24 13:30:23 -07:00
da46e16f4f
command/push: can set the name on the CLI
2015-03-24 13:30:23 -07:00
5e27bfc040
command/push: read name from the config
2015-03-24 13:30:23 -07:00
9062bfda89
command/meta: fix test
2015-03-24 13:30:23 -07:00
eebd7b8aa3
command/push: ask for only variables that are unset
2015-03-24 13:30:23 -07:00
bf14143369
command/push: test that input is asked
2015-03-24 13:30:22 -07:00
d37d9ea6ef
command/push: send the context variables up
2015-03-24 13:30:22 -07:00
a1b424d53f
command/push: properly copy the data directory no matter what
2015-03-24 13:30:22 -07:00
22087181af
command/push: archive, upload
2015-03-24 13:30:22 -07:00
ca8e2085f3
command/push: archiving
2015-03-24 13:30:22 -07:00
c4dc9af120
command: add DataDir
2015-03-24 13:30:21 -07:00
cdde9149ff
command/push: start it
2015-03-24 13:30:21 -07:00
4fc5ebf47e
command: warnings should not exit
...
also properly colorize error/warnings
depends on https://github.com/mitchellh/cli/pull/15
2015-03-05 14:22:34 -06:00
89e7438f0f
command/remote
2015-03-04 16:25:11 -08:00
01cd761023
command: move remote configuration stuff
2015-03-04 16:17:30 -08:00
71315076c3
command: autoload terraform.tfvars.json as well [GH-1030]
2015-03-02 09:22:28 -08:00
8e76a02a56
command: disable backup with "-" [GH-1072]
...
/cc @phinze
2015-03-02 09:11:49 -08:00
6f9a358cc4
command/taint: fix comment
2015-02-26 14:30:02 -08:00
d411e2939f
command/taint: -allow-missing
2015-02-26 10:56:45 -08:00
01aa4236c0
command/taint: support tainting resources in modules
2015-02-26 10:44:25 -08:00
fa9b655fd1
website: docs for tainted command
2015-02-26 10:37:08 -08:00
4ec31ecb95
command/taint: new command
2015-02-26 10:29:23 -08:00
95cf69aa32
command: StateHook for continous state updates
2015-02-23 21:57:17 -08:00
a655c1619a
command: update help text
2015-02-23 15:13:56 -08:00
0299e60e83
command/init: -backend-config
2015-02-23 15:13:56 -08:00
84a0e512d3
command/remote: general `-config` flag
2015-02-23 15:13:56 -08:00
189e7e700a
command: remove remote package
2015-02-23 15:13:55 -08:00
2c2b560d7f
command/remote: no more remote package
2015-02-23 15:13:55 -08:00
0fcc417ddd
command/apply: remove remote package
2015-02-23 15:13:55 -08:00
0c6d133d61
command/refresh: get rid of remote package
2015-02-23 15:13:55 -08:00
0d39a5d9a7
command/push: no more remote package
2015-02-23 15:13:55 -08:00
f2c6c12535
command/pull: remove remote package
2015-02-23 15:13:55 -08:00
b8a66cb6ca
command: remove more remote package
2015-02-23 15:13:55 -08:00
622690583c
command/init: remove dependency on remote package
2015-02-23 15:13:55 -08:00
3550f7ac3a
command: FlagVar => FlagKV
2015-02-23 15:13:54 -08:00
4f91b98a3e
command: default path should be the local path
2015-02-23 15:13:54 -08:00
f81110c2fe
command: backup stuff works better
2015-02-23 15:13:54 -08:00
4ec63bc2ef
command: deal with plan states
2015-02-23 15:13:54 -08:00
579f102f37
command: start migrating to new state package
2015-02-23 15:13:54 -08:00
c2593f6ada
terraform: re-enable dot-graphs
2015-02-19 23:00:29 -08:00
b1e5b32322
terraform: Graph returns *Graph for now
2015-02-19 12:08:32 -08:00
6c8dd9e8fa
Unreachable code removed
2015-02-18 18:28:09 +00:00
92540d3d88
fixing version numbers RCs should be labeled x.x.x-rcx
...
see conversation with ryanuber: https://github.com/hashicorp/go-checkpoint/issues/2#issuecomment-73199209
2015-02-07 16:56:56 +01:00
b40c2fe997
command: fix test failure on Windows
...
URLs are `/`-based. Windows path Separator is `\`.
Convert `\` in test fixture path to `/` with filepath.ToSlash
such that proper URLs are constructed even on Windows.
Fixes a test failure on Windows.
2015-01-28 08:13:44 +01:00
92e68c1ee6
command: fix test failure on Windows
...
Adjust the laddr argument to net.Listen to the form host:port,
as documented for net.Dial.
Fixes a test failure on Windows.
2015-01-28 08:10:52 +01:00
b211d69dde
Fix typo in refresh command error
...
Noticed this while I was digging around to see how the commands work.
2015-01-17 15:43:04 -08:00
fb3f10efb0
command/apply: ask for user variables before validation [GH-736]
2015-01-16 10:46:38 -08:00
90138825a2
command/apply: don't init if no args given [GH-780]
...
I really don't know how to unit test this since I don't know what input
triggers this except that it has been reporting a few times in the wild.
2015-01-16 10:22:20 -08:00
b69239e7f8
command/meta: Adding additional nil guards
2015-01-07 13:28:15 -08:00
408ae62306
command/apply: Handle remote state in a plan file
2015-01-07 13:08:32 -08:00
07505ed159
Fix typo: expacts -> expects
2015-01-03 17:59:55 +01:00
fc36b1cd94
command: vet fix
...
Fixes the following vet report:
command/remote.go:165: wrong number of args for format in Sprintf call: 1 needed but 2 args
2014-12-17 13:02:19 +01:00
cce881aadd
command/show: enable printing of remote state
2014-12-10 13:27:11 -08:00
07e5f3ff41
command/remote: Improve UI output for remote
2014-12-10 13:27:11 -08:00
bcd348aebd
command: updating tests
2014-12-10 13:27:10 -08:00
6a84be0686
command: updating for pluggable backends
2014-12-10 13:27:10 -08:00
ba01e27026
command: Refresh remote state when loading
2014-12-10 13:27:09 -08:00
dab47b0d48
command/plan: remote enable
2014-12-10 13:27:09 -08:00
James Nugent