Merge pull request #69 from pelotech/cleanup-on-fail

Pass --cleanup-on-fail when so instructed

docs/parameter_reference.md

@@ -39,6 +39,7 @@ Installations are triggered when the `helm_command` setting is "upgrade." They c
 | timeout                | duration       |          | Timeout for any *individual* Kubernetes operation. The installation's full runtime may exceed this duration. |
 | force                  | boolean        |          | Pass `--force` to `helm upgrade`. |
 | atomic_upgrade         | boolean        |          | Pass `--atomic` to `helm upgrade`. |
+| cleanup_failed_upgrade | boolean        |          | Pass `--cleanup-on-fail` to `helm upgrade`. |
 | values                 | list\<string\> |          | Chart values to use as the `--set` argument to `helm upgrade`. |
 | string_values          | list\<string\> |          | Chart values to use as the `--set-string` argument to `helm upgrade`. |
 | values_files           | list\<string\> |          | Values to use as `--values` arguments to `helm upgrade`. |

internal/helm/config.go

@@ -44,6 +44,7 @@ type Config struct {
 	Release            string   ``                                   // Release argument to use in applicable helm commands
 	Force              bool     ``                                   // Pass --force to applicable helm commands
 	AtomicUpgrade      bool     `split_words:"true"`                 // Pass --atomic to `helm upgrade`
+	CleanupOnFail      bool     `envconfig:"CLEANUP_FAILED_UPGRADE"` // Pass --cleanup-on-fail to `helm upgrade`
 	LintStrictly       bool     `split_words:"true"`                 // Pass --strict to `helm lint`
 
 	Stdout io.Writer `ignored:"true"`

internal/helm/plan.go

@@ -98,18 +98,19 @@ var upgrade = func(cfg Config) []Step {
 		steps = append(steps, depUpdate(cfg)...)
 	}
 	steps = append(steps, &run.Upgrade{
-		Chart:        cfg.Chart,
+		Chart:         cfg.Chart,
-		Release:      cfg.Release,
+		Release:       cfg.Release,
-		ChartVersion: cfg.ChartVersion,
+		ChartVersion:  cfg.ChartVersion,
-		DryRun:       cfg.DryRun,
+		DryRun:        cfg.DryRun,
-		Wait:         cfg.Wait,
+		Wait:          cfg.Wait,
-		Values:       cfg.Values,
+		Values:        cfg.Values,
-		StringValues: cfg.StringValues,
+		StringValues:  cfg.StringValues,
-		ValuesFiles:  cfg.ValuesFiles,
+		ValuesFiles:   cfg.ValuesFiles,
-		ReuseValues:  cfg.ReuseValues,
+		ReuseValues:   cfg.ReuseValues,
-		Timeout:      cfg.Timeout,
+		Timeout:       cfg.Timeout,
-		Force:        cfg.Force,
+		Force:         cfg.Force,
-		Atomic:       cfg.AtomicUpgrade,
+		Atomic:        cfg.AtomicUpgrade,
+		CleanupOnFail: cfg.CleanupOnFail,
 	})
 
 	return steps

internal/helm/plan_test.go

@@ -142,6 +142,7 @@ func (suite *PlanTestSuite) TestUpgrade() {
 		Release:       "post_malone_circles",
 		Force:         true,
 		AtomicUpgrade: true,
+		CleanupOnFail: true,
 	}
 
 	steps := upgrade(cfg)
@@ -152,18 +153,19 @@ func (suite *PlanTestSuite) TestUpgrade() {
 	upgrade, _ := steps[1].(*run.Upgrade)
 
 	expected := &run.Upgrade{
-		Chart:        cfg.Chart,
+		Chart:         cfg.Chart,
-		Release:      cfg.Release,
+		Release:       cfg.Release,
-		ChartVersion: cfg.ChartVersion,
+		ChartVersion:  cfg.ChartVersion,
-		DryRun:       true,
+		DryRun:        true,
-		Wait:         cfg.Wait,
+		Wait:          cfg.Wait,
-		Values:       "steadfastness,forthrightness",
+		Values:        "steadfastness,forthrightness",
-		StringValues: "tensile_strength,flexibility",
+		StringValues:  "tensile_strength,flexibility",
-		ValuesFiles:  []string{"/root/price_inventory.yml"},
+		ValuesFiles:   []string{"/root/price_inventory.yml"},
-		ReuseValues:  cfg.ReuseValues,
+		ReuseValues:   cfg.ReuseValues,
-		Timeout:      cfg.Timeout,
+		Timeout:       cfg.Timeout,
-		Force:        cfg.Force,
+		Force:         cfg.Force,
-		Atomic:       true,
+		Atomic:        true,
+		CleanupOnFail: true,
 	}
 
 	suite.Equal(expected, upgrade)

internal/run/upgrade.go

@@ -9,16 +9,17 @@ type Upgrade struct {
 	Chart   string
 	Release string
 
-	ChartVersion string
+	ChartVersion  string
-	DryRun       bool
+	DryRun        bool
-	Wait         bool
+	Wait          bool
-	Values       string
+	Values        string
-	StringValues string
+	StringValues  string
-	ValuesFiles  []string
+	ValuesFiles   []string
-	ReuseValues  bool
+	ReuseValues   bool
-	Timeout      string
+	Timeout       string
-	Force        bool
+	Force         bool
-	Atomic       bool
+	Atomic        bool
+	CleanupOnFail bool
 
 	cmd cmd
 }
@@ -69,6 +70,9 @@ func (u *Upgrade) Prepare(cfg Config) error {
 	if u.Atomic {
 		args = append(args, "--atomic")
 	}
+	if u.CleanupOnFail {
+		args = append(args, "--cleanup-on-fail")
+	}
 	if u.Values != "" {
 		args = append(args, "--set", u.Values)
 	}

internal/run/upgrade_test.go

@@ -89,18 +89,19 @@ func (suite *UpgradeTestSuite) TestPrepareWithUpgradeFlags() {
 	defer suite.ctrl.Finish()
 
 	u := Upgrade{
-		Chart:        "hot_ac",
+		Chart:         "hot_ac",
-		Release:      "maroon_5_memories",
+		Release:       "maroon_5_memories",
-		ChartVersion: "radio_edit",
+		ChartVersion:  "radio_edit",
-		DryRun:       true,
+		DryRun:        true,
-		Wait:         true,
+		Wait:          true,
-		Values:       "age=35",
+		Values:        "age=35",
-		StringValues: "height=5ft10in",
+		StringValues:  "height=5ft10in",
-		ValuesFiles:  []string{"/usr/local/stats", "/usr/local/grades"},
+		ValuesFiles:   []string{"/usr/local/stats", "/usr/local/grades"},
-		ReuseValues:  true,
+		ReuseValues:   true,
-		Timeout:      "sit_in_the_corner",
+		Timeout:       "sit_in_the_corner",
-		Force:        true,
+		Force:         true,
-		Atomic:       true,
+		Atomic:        true,
+		CleanupOnFail: true,
 	}
 
 	cfg := Config{}
@@ -115,6 +116,7 @@ func (suite *UpgradeTestSuite) TestPrepareWithUpgradeFlags() {
 			"--timeout", "sit_in_the_corner",
 			"--force",
 			"--atomic",
+			"--cleanup-on-fail",
 			"--set", "age=35",
 			"--set-string", "height=5ft10in",
 			"--values", "/usr/local/stats",