Skip to content

Commit

Permalink
Merge pull request #5348 from r-vasquez/allow-failure-gcp-tuner
Browse files Browse the repository at this point in the history
rpk: allow write_disk_cache to fail in rpk tune
  • Loading branch information
jcsp committed Jul 5, 2022
2 parents 630e791 + ad33f14 commit efba8c7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/go/rpk/pkg/cli/cmd/redpanda/tune.go
Original file line number Diff line number Diff line change
Expand Up @@ -181,8 +181,10 @@ func tune(
supported, reason := tuner.CheckIfSupported()
if !enabled || !supported {
includeErr = includeErr || !supported
exit1 = exit1 || enabled && !supported
results = append(results, result{tunerName, false, enabled, supported, reason})
// We exit with code 1 when it's enabled and not supported except
// for disk_write_cache since it's only supported for GCP.
exit1 = exit1 || enabled && !supported && tunerName != "disk_write_cache"
continue
}
log.Debugf("Tuner parameters %+v", params)
Expand All @@ -192,8 +194,6 @@ func tune(
errMsg := ""
if res.IsFailed() {
errMsg = res.Error().Error()
// We exit with code 1 when it's enabled and not supported
// or when one tuner fails.
exit1 = true
}
results = append(results, result{tunerName, !res.IsFailed(), enabled, supported, errMsg})
Expand Down

0 comments on commit efba8c7

Please sign in to comment.