Skip to content

Conversation

@evanpurkhiser
Copy link
Member

This will be used to inform the clock tick tasks that the tick detected
an abnormal amount of check-in volume for the previous minute.

Part of getsentry/sentry#79328

@evanpurkhiser evanpurkhiser requested a review from a team as a code owner October 23, 2024 19:49
@github-actions
Copy link

github-actions bot commented Oct 23, 2024

versions in use:

The following repositories use one of the schemas you are editing. It is recommended to roll out schema changes in small PRs, meaning that if those used versions lag behind the latest, it is probably best to update those services before rolling out your change.

  • getsentry/sentry: pip:sentry-kafka-schemas==0.1.111 (upgrade)

latest version: 0.1.113

benign changes

schemas/monitors-clock-tick.v1.schema.json

  • Added a new property.

    {"path": "", "change": {"PropertyAdd": {"lhs_additional_properties": false, "added": "volume_anomaly_result"}}}
    

✅ This PR should be safe to roll out to consumers first. Make sure to bump
the library in the following repos first:

getsentry/sentry

...then in the other repos:

getsentry/sentry

Take a look at the README for how to release a new version of sentry-kafka-schemas.

This will be used to inform the clock tick tasks that the tick detected
an abnormal amount of check-in volume for the previous minute.

Part of getsentry/sentry#79328
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-crons-add-volume-detection-result-to-clock-tick branch from 6628f8a to 1794dcf Compare October 23, 2024 20:34
@evanpurkhiser evanpurkhiser changed the title feat(crons): Add "volume_detection" result to clock-tick feat(crons): Add "volume_anomaly_result" result to clock-tick Oct 23, 2024
@evanpurkhiser evanpurkhiser merged commit 306a1d9 into main Oct 24, 2024
17 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-crons-add-volume-detection-result-to-clock-tick branch October 24, 2024 16:31
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.

3 participants