diff options
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/repro/repro.go | 12 | ||||
| -rw-r--r-- | pkg/repro/repro_test.go | 2 |
2 files changed, 10 insertions, 4 deletions
diff --git a/pkg/repro/repro.go b/pkg/repro/repro.go index b774705e6..1da74830f 100644 --- a/pkg/repro/repro.go +++ b/pkg/repro/repro.go @@ -527,13 +527,19 @@ func (ctx *context) testProg(p *prog.Prog, duration time.Duration, opts csource. func (ctx *context) testWithInstance(callback func(execInterface) (rep *instance.RunResult, err error)) (bool, error) { - result, err := ctx.runOnInstance(callback) - if err != nil { + var result *instance.RunResult + var err error + + const attempts = 3 + for i := 0; i < attempts; i++ { // It's hard to classify all kinds of errors into the one worth repeating - // and not. So let's just retry run for all errors. + // and not. So let's just retry runs for all errors. // If the problem is transient, it will likely go away. // If the problem is permanent, it will just be the same. result, err = ctx.runOnInstance(callback) + if err == nil { + break + } } if err != nil { return false, err diff --git a/pkg/repro/repro_test.go b/pkg/repro/repro_test.go index 0e58685b7..fc8d9cad0 100644 --- a/pkg/repro/repro_test.go +++ b/pkg/repro/repro_test.go @@ -247,7 +247,7 @@ func TestTooManyErrors(t *testing.T) { t: t, run: func(log []byte) (*instance.RunResult, error) { counter++ - if counter%3 != 0 { + if counter%4 != 0 { return nil, fmt.Errorf("some random error") } return testExecRunner(log) |