Distillation

Date: Aug 4, 2026

• 🟡 (19:32) PR #21336 body retrieved in full: title fix(orbital): restore notify metrics, IP fallback, and Service DNS. Lists 3 changes: (1) re-emit getsentry.orbital.notify with outcomes no_socket, ip_present, ip_fallback, ip_missing, geo_error, geo_miss, send_error, sent; (2) IP fallback to data.user.ip_address when Kafka remote_addr missing; (3) US ORBITAL_UDP_SERVER default → orbital-udp.default.svc.cluster.local:5556 (replacing stale ILB IP 10.2.0.43 from 2019)
• 🔴 (19:32) PR #21336 notes "prod only receives ~1–2 UDP events/s" even at sample-rate=1.0, and Orbital /stats shows dropped=0 — loss is upstream in notify_orbital (getsentry → orbital publisher)
• 🔴 (19:32) PR #21336 companion action required: ops must set same DNS in US env override file getsentry/ops/k8s/services/getsentry/region_overrides/us/default.yaml, otherwise injected ORBITAL_UDP_SERVER still wins
• 🟡 (19:32) PR #21336 post-deploy monitoring guidance: watch Datadog getsentry.orbital.notify by outcome, and https://live.sentry.io/stats (udp_received rate); sent should rise, if send_error dominates → DNS/routing still needs ops attention
• 🟡 (19:32) Diff output retrieved was massive and contained many unrelated changes (truncated at 3175 chars elided, etc.); assistant noted truncation/irrelevance and recommended checking only the current commit's actual changes
• 🟡 (19:32) New test test_send_error added: patches orbital.udp_socket with sendto.side_effect = OSError("network unreachable"), calls orbital.notify_orbital("1.2.3.4", data={"platform": "python"}), asserts outcomes ["ip_present", "send_error"] (uses _outcomes(incr) helper)
• 🟡 (19:32) test_sent updated to also patch orbital.metrics.incr as incr, calls orbital.notify_orbital("1.2.3.4", data={"platform": "Python"})
• 🟡 (19:32) Unrelated test edits visible in same diff: removed test_free_trial_of_same_family_flag_trial_distinct_from_paid and assertion subscription_on-non-enterprise-trial in TestGetsentryFlagpoleContextBuilder; removed with_feature import in tests/getsentry/jobs/test_update_seat_identifiers_with_detector_ids.py; added PotentialProductTrial import from getsentry.billing.product_trials.data
• 🟡 (19:32) Unrelated test additions visible in same diff: TestReplayIngestionFeatureHandler extended with test_incident_management and other feature handler tests (organizations:data-forwarding, organizations:profiling-view, projects:discard-groups); new SpanDynamicSamplingFeatureHandlerTest class with assert self.handler.has(feature, actor=_as_actor(self.user)) pattern; TestSingleTenantFeatureHandler changes
• 🟡 (19:32) Unrelated service config visible: "path": "sentry.services.eventstore.processing.redis.RedisClusterEventProcessingStore", "host": "127.0.0.1", _init_service_delegator(), processing_store = _init_service_delegator() — appears to be eventstore processing store delegation changes