Skip to content

chore: add kube pod annotation to debug request-replayer#3978

Merged
Leiyks merged 1 commit into
masterfrom
leiyks/set-pod-annotation-on-test_extension_ci
Jun 11, 2026
Merged

chore: add kube pod annotation to debug request-replayer#3978
Leiyks merged 1 commit into
masterfrom
leiyks/set-pod-annotation-on-test_extension_ci

Conversation

@Leiyks

@Leiyks Leiyks commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Description

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@Leiyks
Leiyks requested a review from a team as a code owner June 11, 2026 16:31
@Leiyks
Leiyks merged commit c079885 into master Jun 11, 2026
30 of 31 checks passed
@Leiyks
Leiyks deleted the leiyks/set-pod-annotation-on-test_extension_ci branch June 11, 2026 16:33
@github-actions github-actions Bot added this to the 1.22.0 milestone Jun 11, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 119be3842d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

MAX_TEST_PARALLELISM: 4
PHP_MAJOR_MINOR: "<?= $major_minor ?>"
ARCH: "amd64"
KUBERNETES_POD_ANNOTATIONS_1: "ci.ddbuild.io/enforce-static-cpus=true"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Set a CPU limit when requesting static CPUs

On the Kubernetes runners where ci.ddbuild.io/enforce-static-cpus=true is honored, this job now opts into static CPU allocation while only setting KUBERNETES_CPU_REQUEST; GitLab Runner treats CPU request and limit as separate build-container settings, so the generated pod lacks the matching CPU limit needed for a Guaranteed/static-CPU workload. The other existing static-CPU job in this file sets request and limit together, so add KUBERNETES_CPU_LIMIT: 12 here or drop the static-CPU annotation to avoid these test_extension_ci jobs being rejected or not receiving the intended pinned CPUs.

Useful? React with 👍 / 👎.

bwoebi added a commit that referenced this pull request Jun 12, 2026
…set-ddtrace-retry-interval

* 'master' of github.com:DataDog/dd-trace-php:
  libdatadog update to 6760faae (#3944)
  feat(profiling): add heap-live profiling for memory leak detection (#3623)
  chore: add kube pod annotation to debug request-replayer (#3978)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants