Skip to content

Comments

test(profiling): unflake test_uwsgi_threads_processes_primary_lazy_apps#16605

Open
KowalskiThomas wants to merge 1 commit intomainfrom
kowalski/test-profiling-unflake-test_uwsgi_threads_processes_primary_lazy_apps
Open

test(profiling): unflake test_uwsgi_threads_processes_primary_lazy_apps#16605
KowalskiThomas wants to merge 1 commit intomainfrom
kowalski/test-profiling-unflake-test_uwsgi_threads_processes_primary_lazy_apps

Conversation

@KowalskiThomas
Copy link
Contributor

Description

https://datadoghq.atlassian.net/browse/PROF-13361

This uses the same technique as I did in other uWSGI tests to unflake test_uwsgi_threads_processes_primary_lazy_apps.

@cit-pr-commenter-54b7da
Copy link

Codeowners resolved as

tests/profiling/test_uwsgi.py                                           @DataDog/profiling-python

@KowalskiThomas KowalskiThomas added changelog/no-changelog A changelog entry is not required for this PR. Profiling Continous Profling labels Feb 20, 2026
@KowalskiThomas KowalskiThomas marked this pull request as ready for review February 20, 2026 16:20
@KowalskiThomas KowalskiThomas requested a review from a team as a code owner February 20, 2026 16:20
@taegyunkim
Copy link
Contributor

/merge

@gh-worker-devflow-routing-ef8351
Copy link

gh-worker-devflow-routing-ef8351 bot commented Feb 20, 2026

View all feedbacks in Devflow UI.

2026-02-20 16:20:55 UTC ℹ️ Start processing command /merge


2026-02-20 16:21:02 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-02-20 16:37:19 UTC ⚠️ MergeQueue: This merge request was unqueued

thomas.kowalski@datadoghq.com unqueued this merge request

Copy link

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

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: 32e83c0561

ℹ️ 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".

@KowalskiThomas KowalskiThomas force-pushed the kowalski/test-profiling-unflake-test_uwsgi_threads_processes_primary_lazy_apps branch from 32e83c0 to 82b558f Compare February 20, 2026 16:35
@KowalskiThomas KowalskiThomas force-pushed the kowalski/test-profiling-unflake-test_uwsgi_threads_processes_primary_lazy_apps branch from 82b558f to 9cc7aed Compare February 23, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog A changelog entry is not required for this PR. Profiling Continous Profling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants