Skip to content

chore(integrations): Changes exception type for missing org integrations, updates assignment sync to halt on this exception#91542

Merged
GabeVillalobos merged 2 commits into
masterfrom
gv/reclassify-missing-org-integrations-as-halt
May 15, 2025
Merged

chore(integrations): Changes exception type for missing org integrations, updates assignment sync to halt on this exception#91542
GabeVillalobos merged 2 commits into
masterfrom
gv/reclassify-missing-org-integrations-as-halt

Conversation

@GabeVillalobos

Copy link
Copy Markdown
Member

Fixes ECO-507

Updates the exception type raised in the base IntegrationInstallation class to a distinct type, handles this case when choosing an SLO outcome in sync_assignment_outbound tasks to ensure that missing org integrations are treated as halts, not failures.

…ons, updates assignment sync to halt on this exception
@GabeVillalobos GabeVillalobos requested review from a team as code owners May 13, 2025 16:26
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 13, 2025
@codecov

codecov Bot commented May 13, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #91542       +/-   ##
===========================================
+ Coverage   45.40%   87.64%   +42.23%     
===========================================
  Files       10299    10321       +22     
  Lines      585388   586474     +1086     
  Branches    22602    22602               
===========================================
+ Hits       265804   514013   +248209     
+ Misses     319156    72033   -247123     
  Partials      428      428               

@GabeVillalobos GabeVillalobos merged commit 70c8e92 into master May 15, 2025
60 checks passed
@GabeVillalobos GabeVillalobos deleted the gv/reclassify-missing-org-integrations-as-halt branch May 15, 2025 17:02
@sentry

sentry Bot commented May 15, 2025

Copy link
Copy Markdown
Contributor

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ OrganizationIntegrationNotFound: missing org_integration sentry.integrations.tasks.sync_status_outbound View Issue
  • ‼️ OrganizationIntegrationNotFound: missing org_integration sentry.tasks.process_commit_context View Issue
  • ‼️ OrganizationIntegrationNotFound: missing org_integration sentry.integrations.tasks.sync_status_outbound View Issue
  • ‼️ OrganizationIntegrationNotFound: missing org_integration sentry.tasks.process_commit_context View Issue
  • ‼️ OrganizationIntegrationNotFound: missing org_integration /api/0/projects/{organization_id_or_slug}/{proj... View Issue

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

@github-actions github-actions Bot locked and limited conversation to collaborators Jun 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants