Skip to content

Python: bump misc-integration retry delay to 30s#5293

Merged
moonbox3 merged 1 commit intomicrosoft:mainfrom
moonbox3:improve-int-test-robustness
Apr 16, 2026
Merged

Python: bump misc-integration retry delay to 30s#5293
moonbox3 merged 1 commit intomicrosoft:mainfrom
moonbox3:improve-int-test-robustness

Conversation

@moonbox3
Copy link
Copy Markdown
Contributor

The misc-integration job (Anthropic, Ollama, MCP) frequently fails on merge to main when the upstream MCP server (e.g. learn.microsoft.com/api/mcp) returns a transient rate-limit error surfaced by Anthropic as a 400. The previous 5s --retry-delay is too short to ride out the upstream backoff window, so all pytest-retry attempts fail in quick succession and block the merge queue.

Bumping the misc-integration retry delay to 30s gives the upstream a chance to recover before re-running. Scoped to the misc-integration job in both python-merge-tests.yml and python-integration-tests.yml; other integration lanes are unchanged.

The misc-integration job (Anthropic, Ollama, MCP) frequently fails on merge to main when the upstream MCP server (e.g. learn.microsoft.com/api/mcp) returns a transient rate-limit error. The previous 5s retry delay is too short to ride out the upstream backoff window, so all retries fail and the merge queue is blocked. Bumping to 30s gives the upstream a chance to recover before pytest-retry re-runs the test.
Copilot AI review requested due to automatic review settings April 16, 2026 00:00
@moonbox3 moonbox3 self-assigned this Apr 16, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts GitHub Actions workflow configuration to reduce flakiness in the Python “misc integration” lane (Anthropic/Ollama/MCP) by increasing the delay between pytest-retry attempts, helping transient upstream rate limits/backoff windows clear before reruns.

Changes:

  • Increase pytest retry delay from 5s to 30s for the misc-integration job in python-merge-tests.yml.
  • Increase pytest retry delay from 5s to 30s for the misc-integration job in python-integration-tests.yml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/python-merge-tests.yml Bumps --retry-delay to 30s for misc integration tests to reduce merge-queue failures due to transient upstream rate limits.
.github/workflows/python-integration-tests.yml Applies the same 30s retry delay adjustment to the misc integration job in the integration workflow.

@moonbox3 moonbox3 merged commit f112150 into microsoft:main Apr 16, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants