Skip to content

Fix flaky tornado test by increasing delta tolerance to 0.02#4550

Merged
xrmx merged 3 commits intomainfrom
copilot/fix-flaky-tornado-test-pypy
May 8, 2026
Merged

Fix flaky tornado test by increasing delta tolerance to 0.02#4550
xrmx merged 3 commits intomainfrom
copilot/fix-flaky-tornado-test-pypy

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 7, 2026

Description

test_metrics_concurrent_requests is flaky on pypy/ubuntu — the server-vs-client duration delta for the fast /async request exceeds the 0.01s tolerance due to scheduling jitter. Bumps to 0.02s.

Fixes #4070

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Existing test test_metrics_concurrent_requests with relaxed delta

Does This PR Require a Core Repo Change?

  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

Copilot AI assigned Copilot and xrmx May 7, 2026
Copilot AI linked an issue May 7, 2026 that may be closed by this pull request
The test_metrics_concurrent_requests test was flaky on pypy/ubuntu
because the delta between server and client duration for fast async
requests can exceed 0.01 seconds due to scheduling jitter.

Assisted-by: Claude Sonnet 4

Agent-Logs-Url: https://github.com/open-telemetry/opentelemetry-python-contrib/sessions/a65606a0-612d-49b2-bcda-8b3942ee3e63

Co-authored-by: xrmx <12932+xrmx@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix flaky tornado test on PyPy and Ubuntu Fix flaky tornado test by increasing delta tolerance to 0.02 May 7, 2026
Copilot finished work on behalf of xrmx May 7, 2026 08:22
Copilot AI requested a review from xrmx May 7, 2026 08:22
@xrmx xrmx added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 7, 2026
@xrmx xrmx moved this to Easy to review / merge / close in Python PR digest May 7, 2026
@xrmx xrmx marked this pull request as ready for review May 7, 2026 08:26
@github-project-automation github-project-automation Bot moved this from Easy to review / merge / close to Approved PRs in Python PR digest May 7, 2026
@xrmx xrmx enabled auto-merge (squash) May 8, 2026 13:26
@xrmx xrmx moved this from Approved PRs to Ready for merge in Python PR digest May 8, 2026
@xrmx xrmx merged commit 47a3d9d into main May 8, 2026
752 checks passed
@xrmx xrmx deleted the copilot/fix-flaky-tornado-test-pypy branch May 8, 2026 13:31
@github-project-automation github-project-automation Bot moved this from Ready for merge to Done in Python PR digest May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

flaky tornado test on pypy / ubuntu

3 participants