Skip to content

feat(quartz): Send monitor config from the trigger with check-ins - #6216

Open
wedamija wants to merge 4 commits into
danf/spring-checkin-scheduled-configfrom
danf/quartz-cron-config
Open

wedamija wants to merge 4 commits into
danf/spring-checkin-scheduled-configfrom
danf/quartz-cron-config

Conversation

@wedamija

@wedamija wedamija commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Part of a series adding automatic Crons monitoring for the most-used job schedulers in each SDK. Check-ins for a monitor that doesn't exist yet are dropped, so today each monitor has to be created by hand with its schedule copied from the code. Sending the job's schedule as the monitor config lets Sentry create the monitor and keep it in sync.

SentryJobListener now sends the trigger's schedule with the in-progress check-in. A CronTrigger is sent as a crontab with its time zone, and a SimpleTrigger that repeats forever as an interval. It's on by default, since adding the listener is already an opt-in to monitoring. On upgrade, existing monitors take their schedule and time zone from the trigger. Setting the sentry-upsert-monitor-config job or trigger data key to "false" turns it off for a job.

JobBuilder.newJob(MyJob.class)
    .usingJobData(SentryJobListener.SENTRY_UPSERT_MONITOR_CONFIG_KEY, "false")

Triggers with a calendar, jobs with several triggers, simple triggers that don't repeat forever and cron syntax Sentry doesn't support check in without a config.

Stacked on #6215.
Docs: getsentry/sentry-docs#19777

@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 43e6900

@sentry

sentry Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.59.0 (1) release

⚙️ sentry-android Build Distribution Settings

Co-Authored-By: Claude <noreply@anthropic.com>
@wedamija
wedamija force-pushed the danf/spring-checkin-scheduled-config branch from ff1e1e2 to 5d745c7 Compare October 5, 2026 22:54
@wedamija
wedamija force-pushed the danf/quartz-cron-config branch from d27bf6b to 44ad1c7 Compare October 5, 2026 22:54
wedamija and others added 2 commits October 5, 2026 16:09
Co-Authored-By: Claude <noreply@anthropic.com>
…nfig' into danf/quartz-merge-tmp

# Conflicts:
#	CHANGELOG.md
@wedamija
wedamija marked this pull request as ready for review October 5, 2026 23:38

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 045b6d8. Configure here.

Comment thread sentry-quartz/src/main/java/io/sentry/quartz/SentryJobListener.java
Co-Authored-By: Claude <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant