lourenco/mgmt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

engine: graph, metaparams: Default rewatch to false

Browse Source 
The frequency of graph changes makes it unlikely that you want this
enabled on most resources by default.
This commit is contained in:
James Shubin
2023-09-01 19:02:11 -04:00
parent 9ac8d7ec49
commit 07bd8afc4a
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
engine/metaparams.go
View File

@@ -37,7 +37,7 @@ var DefaultMetaParams = &MetaParams{
Limit: rate.Inf, // defaults to no limit Limit: rate.Inf, // defaults to no limit
Burst: 0, // no burst needed on an infinite rate Burst: 0, // no burst needed on an infinite rate
//Sema: []string{}, //Sema: []string{},
Rewatch: true, Rewatch: false,
Realize: false, // true would be more awesome, but unexpected for users Realize: false, // true would be more awesome, but unexpected for users
} }
@@ -90,7 +90,9 @@ type MetaParams struct {
// and add it back as a new vertex, thus causing it to run again. This // and add it back as a new vertex, thus causing it to run again. This
// is different from the Retry metaparam which applies during the normal // is different from the Retry metaparam which applies during the normal
// execution. It is only when this is exhausted that we're in permanent // execution. It is only when this is exhausted that we're in permanent
// worker failure, and only then can we rely on this metaparam. // worker failure, and only then can we rely on this metaparam. This is
// false by default, as the frequency of graph changes makes it unlikely
// that you wanted this enabled on most resources.
Rewatch bool `yaml:"rewatch"` Rewatch bool `yaml:"rewatch"`
// Realize ensures that the resource is guaranteed to converge at least // Realize ensures that the resource is guaranteed to converge at least