getsentry / sentry-kafka-schemas

https://getsentry.github.io/sentry-kafka-schemas/
Other
4 stars 1 forks source link

fix(uptime): Add `subscription_id` and remove monitor related ids #308

Closed wedamija closed 3 months ago

wedamija commented 3 months ago

We don't want this system to be aware of the details of which system is consuming it, so just use a subscription id to keep track of things

github-actions[bot] commented 3 months ago

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.

latest version: 0.1.92

changes considered breaking **schemas/uptime-results.v1.schema.json** - Added a new property, but the consumer has been ignoring additional properties so far. This is probably still fine, but please double-check that the producer does not already send this property with a different type in practice than you defined in this schema. ``` {"path": "", "change": {"PropertyAdd": {"lhs_additional_properties": true, "added": "subscription_id"}}} ``` - ``` {"path": "", "change": {"RequiredAdd": {"property": "subscription_id"}}} ```
benign changes **schemas/uptime-results.v1.schema.json** - Removed a property `monitor_environment_id` from ``, but it is still accepted via `additionalProperties=true` ``` {"path": "", "change": {"PropertyRemove": {"lhs_additional_properties": true, "removed": "monitor_environment_id"}}} ``` - Removed a property `monitor_id` from ``, but it is still accepted via `additionalProperties=true` ``` {"path": "", "change": {"PropertyRemove": {"lhs_additional_properties": true, "removed": "monitor_id"}}} ``` - ``` {"path": "", "change": {"RequiredRemove": {"property": "monitor_environment_id"}}} ``` - ``` {"path": "", "change": {"RequiredRemove": {"property": "monitor_id"}}} ```

⚠️ This PR contains breaking changes. Normally you should avoid that and make your consumer backwards-compatible (meaning that updated consumers can still accept old messages). There are a few exceptions: