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

Stacked on #6215 (Spring), which is stacked on #6217 (core MonitorConfigUtils). Merge those first.

📜 Description

SentryJobListener now sends the job's schedule with the in-progress check-in, so Sentry creates or updates the monitor from code. A CronTrigger is sent as a crontab with the trigger's time zone; a SimpleTrigger that repeats forever is sent as an interval.

This is on by default. Set the sentry-upsert-monitor-config job or trigger data key (SentryJobListener.SENTRY_UPSERT_MONITOR_CONFIG_KEY) to "false" to turn it off for a job.

Schedules that can't be represented exactly (triggers with a calendar, jobs with several triggers, simple triggers that don't repeat forever, cron syntax Sentry doesn't support) send no config rather than a wrong one.

💡 Motivation and Context

Check-ins for monitors that don't exist in Sentry are dropped, so users create each monitor by hand. The trigger already has the schedule.

Features are usually opt-in, but adding the listener is already an explicit opt-in to monitoring, so this is on by default. On upgrade, existing monitors get their schedule and time zone from the trigger; set the key to "false" to keep managing them in Sentry.

💚 How did you test it?

SentryJobListenerTest. The conversion is tested in #6217.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

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