-
Notifications
You must be signed in to change notification settings - Fork 715
fix: self-org-trigger issues (release branch) #9350
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
Conversation
Signed-off-by: Yashodhan Joshi <[email protected]>
|
Failed to generate code suggestions for PR |
Greptile OverviewGreptile SummaryCherry-picked multiple bug fixes and improvements from the development branch to the release branch, addressing self-org-trigger functionality, Docker build consistency, and various UI issues. Key ChangesSelf-Reporting Trigger Fixes (Rust)
Docker Build Improvements
UI/UX Improvements
Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant Client
participant ReportingQueue
participant Worker as Worker Thread
participant Ingestion as Ingestion Module
participant OrgCheck as Organization Service
participant DB as Database/Stream
Client->>ReportingQueue: Enqueue Trigger Data
ReportingQueue->>Worker: Message via Channel
Worker->>Worker: Buffer until batch size/timeout
Worker->>Worker: Separate triggers, usages, errors
alt usage_reporting_mode != "remote"
Worker->>Ingestion: ingest_usages(usages)
Ingestion->>DB: Store in META_ORG
end
alt Additional Reporting Orgs Configured
loop For each additional org
Worker->>Ingestion: ingest_reporting_data(triggers)
Ingestion->>DB: Store triggers in org stream
end
end
alt usage_report_to_own_org enabled
Worker->>Worker: Group triggers by org
loop For each trigger org
alt Cloud feature enabled
Worker->>OrgCheck: is_org_in_free_trial_period(org_id)
OrgCheck-->>Worker: trial status
alt Not in trial
Worker->>Worker: Skip this org
end
end
Worker->>Ingestion: ingest_reporting_data(org triggers)
Ingestion->>DB: Store in own org stream
end
end
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
14 files reviewed, no comments
cherry-pick of #9348 for release branch