Skip to content

Commit

Permalink
fix(chart): set max builds to -1 by default (#589)
Browse files Browse the repository at this point in the history
* fix(chart): set max builds to -1 by default

A recent fix standardized on maxBuilds=0 meaning no builds are kept. But the chart sets maxBuilds=0 by default. This changes the default to -1.

* Fixed doc string, which was also wrong.
  • Loading branch information
technosophos committed Aug 13, 2018
1 parent 89b790e commit fb0c2a0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions charts/brigade/values.yaml
Expand Up @@ -146,10 +146,10 @@ vacuum:
age: "720h"
# maxBuilds tells the vacuum what the absolute maximum number of builds may be stored
# at a time. Where possible, we recommend using age rather than builds.
# 0 means no limit is imposed.
# -1 means no limit is imposed.
#
# If both age and maxBuilds are provided, age is applied first, then maxBuilds.
maxBuilds: 0
maxBuilds: -1
serviceAccount:
create: true
name:
Expand Down

0 comments on commit fb0c2a0

Please sign in to comment.