Skip to content

Backlog/v12 datasources log timeline - #2403

Merged
AlexSanchez-bit merged 4 commits into
release/v12.0.0from
backlog/v12_datasources_log_timeline
Jul 28, 2026
Merged

Backlog/v12 datasources log timeline#2403
AlexSanchez-bit merged 4 commits into
release/v12.0.0from
backlog/v12_datasources_log_timeline

Conversation

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor

No description provided.

@AlexSanchez-bit AlexSanchez-bit linked an issue Jul 28, 2026 that may be closed by this pull request
3 tasks
@github-actions

github-actions Bot commented Jul 28, 2026

Copy link
Copy Markdown

🛑 AI review — High/critical findings

One or more high/critical issues were found. Please review and fix before merging if they're real.

⚠️ architecture (gemini-3-flash-lite) — minor findings

Summary: Update AWS SDK dependencies, add time range filtering to datasources ingestion timeline, and adjust time window resolution.

  • low backend/modules/eventprocessing/usecase/ingestion_stats.go:133 — The time window resolution logic now assigns raw strings like 'now' or 'now-' directly. Ensure that downstream parsers or queries correctly support relative time expressions to prevent query execution errors.

🛑 bugs (gemini-3-flash-lite) — high/critical — please review

Summary: Broken Go implementation in resolveWindow using invalid time query strings

  • high backend/modules/eventprocessing/usecase/ingestion_stats.go:134 — resolveWindow replaces proper UTC parsing with raw string literals like "now" and "now-24h0m0s", which will not be parsed correctly as timestamps by underlying database or date-histogram queries, breaking the ingestion timeline API.

security (gemini-3-flash-lite) — clean

Summary: Routine dependency version bumps and addition of time range filtering for ingestion stats.

No findings.

🟢 go-deps — up to date

No pending Go dependency updates.

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor Author

semantic time ranges are supported by opensearch, time window processor was limiting those time ranges

@AlexSanchez-bit
AlexSanchez-bit merged commit d7b70d8 into release/v12.0.0 Jul 28, 2026
1 check passed
@AlexSanchez-bit
AlexSanchez-bit deleted the backlog/v12_datasources_log_timeline branch July 28, 2026 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v12 Ingestion metric panel lacks time range filter controls

1 participant