Skip to content

Commit

Permalink
Remove uses of the word "sane" (#1731)
Browse files Browse the repository at this point in the history
  • Loading branch information
angrycub authored Mar 31, 2023
1 parent eb79206 commit 3d59f1b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -482,7 +482,7 @@ BUG FIXES:

BREAKING CHANGES:

* Retry now has a sane maximum default. Previous versions of Consul Template
* Retry now has a maximum default. Previous versions of Consul Template
would retry indefinitely, potentially allowing the time between retries to
reach days, months, or years due to the exponential nature. Users wishing
to use the old behavior should set `max_backoff = 0` in their
Expand Down
6 changes: 3 additions & 3 deletions manager/runner.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,9 @@ import (
)

const (
// saneViewLimit is the number of views that we consider "sane" before we
// viewLimit is the number of views that we consider reasonable before we
// warn the user that they might be DDoSing their Consul cluster.
saneViewLimit = 128
viewLimit = 128
)

// Runner responsible rendering Templates and invoking Commands.
Expand Down Expand Up @@ -276,7 +276,7 @@ func (r *Runner) Start() {

for {
// Warn the user if they are watching too many dependencies.
if r.watcher.Size() > saneViewLimit {
if r.watcher.Size() > viewLimit {
log.Printf("[WARN] (runner) watching %d dependencies - watching this "+
"many dependencies could DDoS your servers", r.watcher.Size())
} else {
Expand Down

0 comments on commit 3d59f1b

Please sign in to comment.