Skip to content

Integration Errored Notification, Backfill Integration Metadata, Fix DomainScan Assets - #2649

Merged
matoszz merged 1 commit into
mainfrom
fix-galalistenerfailurelogs
Jul 31, 2026
Merged

Integration Errored Notification, Backfill Integration Metadata, Fix DomainScan Assets#2649
matoszz merged 1 commit into
mainfrom
fix-galalistenerfailurelogs

Conversation

@matoszz

@matoszz matoszz commented Jul 30, 2026

Copy link
Copy Markdown
Member

Changes + logging updates in an attempt to fix at least 5+ observed errors actively occurring in production related to integrations (and being reported in #alerts-sink)

  • Dropped the asset_name_owner_id unique index (schema + migration + regen); domain_scan_import now sets source_identifier on create, matches on it before name, and stamps it onto name-matched rows. Removes the duplicate key error and makes scan-import and asset_sync converge on one row
  • Fixed durable log identity; emit sites use intobvs.WithContext; new WithInstallation + owner_id field, applied in Reconcile and both execution entry points; gala's listener-failure line now carries envelope_properties. Fixes reconcile failures logging with no tenant or integration references
  • cloudflare asset-sync/scan-poll/scan-submit now wrap the upstream error with %w instead of discarding it, plus account/page/cursor context
  • googledrive and googleworkspace log why config/metadata was missing and stop swallowing decode errors; upsert conflicts log the lookup key vs record name; registry misses log every registered definition id
  • googledrive folderId required; added jsonschema:"required", validatePayload checks an absent payload as {} so required fires, and StartIntegrationAuth refuses to issue Drive scopes until it validates (new ErrIntegrationUserInputRequired).
  • Misconfig signalling + scan status -> startup backfill flags connected installs failing user-input validation as ERRORED with an org notification
  • inalizeDomainScan now takes the terminal status so a failed scan can't be reported completed

@matoszz
matoszz force-pushed the fix-galalistenerfailurelogs branch from 2561347 to 8b946cd Compare July 31, 2026 15:33
@github-actions github-actions Bot removed the cli label Jul 31, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
5.6% Coverage on New Code (required ≥ 20%)

See analysis details on SonarQube Cloud

@matoszz
matoszz merged commit f2e165b into main Jul 31, 2026
21 of 22 checks passed
@matoszz
matoszz deleted the fix-galalistenerfailurelogs branch July 31, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants