ResiLien
/
terraform
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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.
This commit is contained in:
Martin Atkins 2015-09-04 08:07:16 -07:00
parent c777f8c95b
commit 7c56550633
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
command/plan.go
View File

@ -136,9 +136,9 @@ func (c *PlanCommand) Run(args []string) int {
c.Ui.Output(c.Colorize().Color(fmt.Sprintf( c.Ui.Output(c.Colorize().Color(fmt.Sprintf(
"[reset][bold]Plan:[reset] "+ "[reset][bold]Plan:[reset] "+
"%d to add, %d to change, %d to destroy.", "%d to add, %d to change, %d to destroy.",
countHook.ToAdd, countHook.ToAdd+countHook.ToRemoveAndAdd,
(countHook.ToChange + countHook.ToRemoveAndAdd), countHook.ToChange,
countHook.ToRemove))) countHook.ToRemove+countHook.ToRemoveAndAdd)))
if detailed { if detailed {
return 2 return 2