Skip to content

[Gemini Log Analyzer] Part 1: Backend Foundation (Models, Config, & Setup)#5042

Merged
auto-submit[bot] merged 4 commits intoflutter:mainfrom
ievdokdm:gemini-log-analyzer-1-infra
May 5, 2026
Merged

[Gemini Log Analyzer] Part 1: Backend Foundation (Models, Config, & Setup)#5042
auto-submit[bot] merged 4 commits intoflutter:mainfrom
ievdokdm:gemini-log-analyzer-1-infra

Conversation

@ievdokdm
Copy link
Copy Markdown
Contributor

@ievdokdm ievdokdm commented May 4, 2026

This is Part 1 of a stacked PR series implementing the Gemini Log Analyzer feature: flutter/flutter#185700

This PR establishes the required foundation:

  • Adds APP_DART_GEMINI_LOG_ANALYZER_KEY secret config property to Config.
  • Adds log_analysis and build_id fields to Firestore PresubmitJob model.
  • Adds build_id to PresubmitJobResponse RPC model in cocoon_common.

Stacked PR Series:

  • Part 1: Backend Foundation (This PR)
  • Part 2: Backend Core API
  • Part 3: Frontend Dashboard UI

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 4, 2026
@ievdokdm ievdokdm force-pushed the gemini-log-analyzer-1-infra branch 2 times, most recently from 44379a0 to bed1895 Compare May 4, 2026 20:58
@ievdokdm ievdokdm added CICD Run CI/CD and removed CICD Run CI/CD labels May 4, 2026
@ievdokdm ievdokdm added CICD Run CI/CD and removed CICD Run CI/CD labels May 5, 2026
Copy link
Copy Markdown
Contributor

@eyebrowsoffire eyebrowsoffire left a comment

Choose a reason for hiding this comment

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

LGTM

@ievdokdm ievdokdm added the autosubmit Merge PR when tree becomes green via auto submit App. label May 5, 2026
@auto-submit auto-submit Bot merged commit 348ac7e into flutter:main May 5, 2026
22 checks passed
auto-submit Bot pushed a commit that referenced this pull request May 5, 2026
… & Database Flow) (#5045)

This is Part 2 of the stacked Pull Request series implementing the Gemini Log Analyzer feature.

**Depends on Part 1 (already merged):** #5042

This PR extracts the backend data flow, configuration, dynamic config flags, and serialization models from the API implementation branch into a clean infrastructure layer:
- Configures AppEngine service config and flags in `config.yaml` and generated dynamic config files.
- Implements database storage interface `storeLogAnalysis` in `UnifiedCheckRun`.
- Adds the `logAnalysis` field to the `PresubmitJobResponse` RPC models to serialize results.

### Stacked PR Series:
- Part 1: Backend Foundation (Models & Setup) (Merged)
- **Part 2: Backend Infrastructure (Config & Database Data Flow)** (This PR)
- Part 3: Backend Core API (Genkit Logic & Handlers)
- Part 4: Frontend Dashboard UI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App. CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants