Skip to content

Commit 2f57577

Browse files
authored
Disable legacy Gitlab exec in order to fix jobs failing with green status (#7223)
## Summary of changes Disables `FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY` that we used before to ensure build containers have QoS guaranteed. As of now, it doesn't seem to be necessary anymore, but leads to problems like https://gitlab.com/gitlab-org/gitlab-runner/-/issues/4119. ## Reason for change Apparently it there were issues with Gitlab CI jobs silently randomly failing with a green status. ## Implementation details Delete usages of `FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY` and `KUBERNETES_SERVICE_ACCOUNT_OVERWRITE` ## Test coverage This is the test ## Other details I based this email on similar PRs raised by @ddyurchenko e.g. - DataDog/dd-trace-cpp#225 - DataDog/dd-trace-js#6097
1 parent 7f6d148 commit 2f57577

3 files changed

Lines changed: 0 additions & 13 deletions

File tree

.gitlab/benchmarks/dsm-throughput.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,5 @@ dsm_throughput:
4444
paths:
4545
- artifacts/
4646
expire_in: 3 months
47-
variables:
48-
KUBERNETES_SERVICE_ACCOUNT_OVERWRITE: dd-trace-dotnet
49-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
5047
rules:
5148
- when: on_success

.gitlab/benchmarks/macrobenchmarks.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ update-bp-infra:
6060
rules:
6161
- when: manual
6262
variables:
63-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
6463
AWS_REGION: "us-east-1"
6564
CLEANUP: "false"
6665
AWS_EPHEMERAL_INFRA_PROFILE_SSM_PARAMETER: "ci.dd-trace-dotnet.ephemeral-infra-ci.dd-trace-dotnet-profile"
@@ -98,8 +97,6 @@ update-bp-infra:
9897
- artifacts/
9998
expire_in: 3 months
10099
variables:
101-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
102-
103100
K6_OPTIONS_WARMUP_RATE: 14000
104101
K6_OPTIONS_WARMUP_DURATION: 3m
105102
K6_OPTIONS_WARMUP_GRACEFUL_STOP: 10s
@@ -282,8 +279,6 @@ profiler_cpu_timer_create:
282279
- artifacts/
283280
expire_in: 3 months
284281
variables:
285-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
286-
287282
K6_OPTIONS_WARMUP_RATE: 14000
288283
K6_OPTIONS_WARMUP_DURATION: 3m
289284
K6_OPTIONS_WARMUP_GRACEFUL_STOP: 10s
@@ -472,8 +467,6 @@ profiler_cpu_timer_create-arm64:
472467
- artifacts/
473468
expire_in: 3 months
474469
variables:
475-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
476-
477470
# Whether to cleanup ephemeral instances after benchmarks are run
478471
CLEANUP: "true"
479472

.gitlab/benchmarks/microbenchmarks.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ update-bp-infra:
3838
rules:
3939
- when: manual
4040
variables:
41-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
4241
AWS_REGION: "us-east-1"
4342
CLEANUP: "false"
4443
AWS_EPHEMERAL_INFRA_PROFILE_SSM_PARAMETER: "ci.dd-trace-dotnet.ephemeral-infra-ci.dd-trace-dotnet-profile"
@@ -69,7 +68,6 @@ run-benchmarks:
6968
rules:
7069
- when: on_success
7170
variables:
72-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
7371
AWS_REGION: "us-east-1"
7472
CLEANUP: "false"
7573
AWS_EPHEMERAL_INFRA_PROFILE_SSM_PARAMETER: "ci.dd-trace-dotnet.ephemeral-infra-ci.dd-trace-dotnet-profile"
@@ -101,7 +99,6 @@ upload-to-bp-ui:
10199
rules:
102100
- when: manual
103101
variables:
104-
FF_USE_LEGACY_KUBERNETES_EXECUTION_STRATEGY: "true"
105102
AWS_REGION: "us-east-1"
106103
CLEANUP: "false"
107104
AWS_EPHEMERAL_INFRA_PROFILE_SSM_PARAMETER: "ci.dd-trace-dotnet.ephemeral-infra-ci.dd-trace-dotnet-profile"

0 commit comments

Comments
 (0)