Skip to content

Retry transient GitHub config fetch failures#98

Draft
cursor[bot] wants to merge 3 commits into
mainfrom
cursor/sentry-error-investigation-ecac
Draft

Retry transient GitHub config fetch failures#98
cursor[bot] wants to merge 3 commits into
mainfrom
cursor/sentry-error-investigation-ecac

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented Apr 29, 2026

Summary

  • Add bounded retry handling and a timeout when fetching .sentry/sentry_config.ini from the GitHub contents API.
  • Retry transient connection errors, timeouts, and retryable GitHub 5xx responses while preserving persistent failures.
  • Add regression coverage for a connection reset followed by a successful config fetch.

Validation

  • python3 -m pytest tests/test_sentry_config_file.py -q
  • python3 -m pytest -q

Fixes SENTRY-GITHUB-ACTIONS-APP-63

Open in Web View Automation 

cursoragent and others added 3 commits April 29, 2026 07:02
Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>
Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>
Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>
Comment thread src/sentry_config.py
Comment on lines +25 to +29
resp = requests.get(
api_url,
headers=headers,
timeout=GITHUB_API_TIMEOUT_SECONDS,
)
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