Skip to content

fix(seer): Scan only new issues or issues without scans #93867

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 18, 2025

Conversation

roaga
Copy link
Member

@roaga roaga commented Jun 18, 2025

Only allow the scanner job to start if an issue is new or does not have an existing scan + summary.

Copy link

sentry-io bot commented Jun 18, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: src/sentry/seer/issue_summary.py

Function Unhandled Issue
get_issue_summary HTTPError: 500 Server Error: Internal Server Error for url: http://seer-web-autofix/v1/automation/summarize/issue ...
Event Count: 35

Did you find this useful? React with a 👍 or 👎

@roaga roaga requested a review from kddubey June 18, 2025 22:07
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 18, 2025
@roaga roaga marked this pull request as ready for review June 18, 2025 22:10
@roaga roaga requested review from a team as code owners June 18, 2025 22:10
Copy link
Member

@JoshFerge JoshFerge left a comment

Choose a reason for hiding this comment

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

i think it's worth writing a quick test for this

@roaga roaga enabled auto-merge (squash) June 18, 2025 22:31
@roaga roaga merged commit 81e2427 into master Jun 18, 2025
63 checks passed
@roaga roaga deleted the seer/scan-new-issues-only branch June 18, 2025 22:40
andrewshie-sentry pushed a commit that referenced this pull request Jun 19, 2025
Only allow the scanner job to start if an issue is new or does not have
an existing scan + summary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants