Skip to content

feat: more debug logging when Watch retries#627

Merged
miparnisari merged 1 commit intomainfrom
logging-watch
Feb 9, 2026
Merged

feat: more debug logging when Watch retries#627
miparnisari merged 1 commit intomainfrom
logging-watch

Conversation

@miparnisari
Copy link
Contributor

Description

Companion to authzed/spicedb#2888

Testing

  1. On tab 1, start SpiceDB
  2. On tab 2, zed watch --log-level=debug
  3. Kill SpiceDB
  4. You should see this in zed:
{"level":"debug","error":"rpc error: code = Unavailable desc = closing transport due to: connection error: desc = \"error reading from server: EOF\", received prior goaway: code: NO_ERROR, debug data: \"graceful_stop\"","time":"2026-02-09T12:17:54-03:00","message":"error receiving from Watch stream"}

{"level":"debug","time":"2026-02-09T12:17:54-03:00","message":"will retry from the last known revision GioKHjE3NzA2NTAyNjgxNDg3MTYwOTYuMDAwMDAwMDAwMBIIYWU5MzAzOTk="}

{"level":"error","error":"error retrying the Watch call: rpc error: code = Unavailable desc = connection error: desc = \"error reading server preface: read tcp 127.0.0.1:64100->127.0.0.1:50051: read: connection reset by peer\"","time":"2026-02-09T12:17:54-03:00","message":"terminated with errors"}

@miparnisari miparnisari marked this pull request as ready for review February 9, 2026 15:20
Copy link
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:GT<

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.72%. Comparing base (fc49cf5) to head (b910015).

Files with missing lines Patch % Lines
internal/commands/watch.go 50.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #627   +/-   ##
=======================================
  Coverage   41.71%   41.72%           
=======================================
  Files          38       38           
  Lines        6130     6131    +1     
=======================================
+ Hits         2557     2558    +1     
  Misses       3317     3317           
  Partials      256      256           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@miparnisari miparnisari changed the title feat: more logging in Watch feat: more debug logging when Watch retries Feb 9, 2026
@miparnisari miparnisari merged commit 56e9ecd into main Feb 9, 2026
10 of 11 checks passed
@miparnisari miparnisari deleted the logging-watch branch February 9, 2026 15:24
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