Distillation

Date: Aug 4, 2026

• 🟡 (19:32) PR #21336 found: fix(orbital): restore notify metrics, IP fallback, and Service DNS (commit 6822013f33) at https://github.com/getsentry/getsentry/pull/21336, state OPEN
• 🟡 (19:32) CI status for PR #21336 (run 30733760389): Backend fail (3s), backend typing fail (1m47s), backend test (1)/(2)/(6) all pass, Cursor Bugbot skipping (3m10s), most other checks skipping. Security/warden checks pass: Socket Security Project Report, warden: security-review, warden: wrdn-code-execution, warden: wrdn-pii
• 🔴 (19:32) In getsentry/conf/settings/cellsilo.py, ORBITAL_UDP_SERVER default changed from 10.2.0.43:5556 (stale Internal LB IP hardcoded since 2019) to orbital-udp.default.svc.cluster.local:5556 (in-cluster Service DNS), inside if SENTRY_LOCAL_CELL == "us":
• 🔴 (19:32) New helper _ip_from_event(data) -> str | None added in getsentry/receivers/orbital.py that falls back to data["user"]["ip_address"] when Kafka remote_addr is missing; validates data is dict, user is dict, ip is non-empty str
• 🔴 (19:32) In notify_orbital(ip, data=None, **kwargs) in getsentry/receivers/orbital.py: when udp_socket is None, now emits metrics.incr("getsentry.orbital.notify", tags={"outcome": "no_socket"}) with comment "Publisher never initialized (ORBITAL_UDP_SERVER unset/misconfigured)"
• 🟡 (19:32) Inline review comments on PR #21336 reference files: getsentry/conf/settings/cellsilo.py (cellsilo.py edit) and getsentry/receivers/orbital.py (notify_orbital + _ip_from_event additions). Reviewers include user BYK (id 126780, github.com/BYK)
• 🟡 (19:32) Assistant reasoning noted review concerns to investigate: DNS name missing trailing dot, metric noise outside US cell (DE region), BYK's "Not sure if this is valid" comment on DNS change, suggestion to define notify_orbital conditionally