Skip to content

Conversation

@detj
Copy link
Contributor

@detj detj commented Oct 28, 2025

Summary

This PR addresses incorrect scheduling of jobs in alerts service. Additionally, improves error handling and logging.

Tasks

  • Use the correct cron syntax for scheduling daily summary job
  • Use the correct cron syntax for scheduling hourly crash & ANR discovery job
  • Improve error handling while scheduling jobs
  • Improve logging while scheduling jobs

See also

detj added 2 commits October 28, 2025 17:58
- use a different timing scheme for cloud that seems to work
- fallback to previous timing scheme for non-cloud environments

fixes #2853

Signed-off-by: detj <[email protected]>
@detj detj self-assigned this Oct 28, 2025
@detj detj added bug something isn't working backend backend related labels Oct 28, 2025
@detj detj moved this to In Progress in Measure Roadmap Oct 28, 2025
@vercel
Copy link

vercel bot commented Oct 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
measure-dashboard Ready Ready Preview Comment Oct 29, 2025 4:49am

- fix cron syntax for daily summary job
- fix cron syntax for crash & ANR job
- improve error handling
- improve logging

fixes #2853

Signed-off-by: detj <[email protected]>
@detj detj changed the title fix(backend): use runtime specific alert timing scheme fix(backend): scheduling of daily & hourly jobs Oct 29, 2025
@detj detj changed the title fix(backend): scheduling of daily & hourly jobs fix(backend): scheduling of daily & hourly alert jobs Oct 29, 2025
@detj detj requested a review from abhaysood October 29, 2025 05:20
@detj detj merged commit 09d286b into main Oct 29, 2025
17 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Measure Roadmap Oct 29, 2025
@detj detj deleted the cloud-notifs-summaries branch October 29, 2025 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend backend related bug something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Fix incorrect cron scheduling in alerts

3 participants