Commit f049bb62 authored by Shinya Maeda's avatar Shinya Maeda

Do not auto-retry unrecoverable job failure

Currently, the system tries to auto-retry jobs
that failed by 'environment_creation_failure',
however, this failure is not recoverable.

This commit force-overrides the auto-retry
maximum on the specific failure.

Changelog: fixed
parent 82e7eda0
...@@ -11,7 +11,8 @@ class Gitlab::Ci::Build::AutoRetry ...@@ -11,7 +11,8 @@ class Gitlab::Ci::Build::AutoRetry
ci_quota_exceeded: 0, ci_quota_exceeded: 0,
no_matching_runner: 0, no_matching_runner: 0,
missing_dependency_failure: 0, missing_dependency_failure: 0,
forward_deployment_failure: 0 forward_deployment_failure: 0,
environment_creation_failure: 0
}.freeze }.freeze
def initialize(build) def initialize(build)
......
...@@ -26,6 +26,7 @@ RSpec.describe Gitlab::Ci::Build::AutoRetry do ...@@ -26,6 +26,7 @@ RSpec.describe Gitlab::Ci::Build::AutoRetry do
"no matching runner" | 0 | { max: 2 } | :no_matching_runner | false "no matching runner" | 0 | { max: 2 } | :no_matching_runner | false
"missing dependencies" | 0 | { max: 2 } | :missing_dependency_failure | false "missing dependencies" | 0 | { max: 2 } | :missing_dependency_failure | false
"forward deployment failure" | 0 | { max: 2 } | :forward_deployment_failure | false "forward deployment failure" | 0 | { max: 2 } | :forward_deployment_failure | false
"environment creation failure" | 0 | { max: 2 } | :environment_creation_failure | false
end end
with_them do with_them do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment