Commit 73273585 authored by Shinya Maeda's avatar Shinya Maeda

Do not auto-retry unrecoverable `forward_deployment_failure` job failure

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

This commit force-overrides the auto-retry
maximum on the specific failure.
parent 4114f576
...@@ -10,7 +10,8 @@ class Gitlab::Ci::Build::AutoRetry ...@@ -10,7 +10,8 @@ class Gitlab::Ci::Build::AutoRetry
RETRY_OVERRIDES = { RETRY_OVERRIDES = {
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
}.freeze }.freeze
def initialize(build) def initialize(build)
......
...@@ -25,6 +25,7 @@ RSpec.describe Gitlab::Ci::Build::AutoRetry do ...@@ -25,6 +25,7 @@ RSpec.describe Gitlab::Ci::Build::AutoRetry do
"quota is exceeded" | 0 | { max: 2 } | :ci_quota_exceeded | false "quota is exceeded" | 0 | { max: 2 } | :ci_quota_exceeded | false
"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
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