Skip to content

Conversation

gregns1
Copy link
Contributor

@gregns1 gregns1 commented Jul 28, 2025

CBG-4757

  • Added some debug logging around reconnect
  • Added debug logging when we successfully reconnect
  • Adds config ID to the error case logging

Open to ideas about anything else that needs adding.

Pre-review checklist

  • Removed debug logging (fmt.Print, log.Print, ...)
  • Logging sensitive data? Make sure it's tagged (e.g. base.UD(docID), base.MD(dbName))
  • Updated relevant information in the API specifications (such as endpoint descriptions, schemas, ...) in docs/api

Dependencies (if applicable)

  • Link upstream PRs
  • Update Go module dependencies when merged

Integration Tests

@Copilot Copilot AI review requested due to automatic review settings July 28, 2025 12:08
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Enhances debugging capabilities for ISGR (Inter Sync Gateway Replication) reconnection logic by adding detailed logging around connection attempts. The PR focuses on improving troubleshooting by providing more visibility into reconnection processes and outcomes.

  • Added debug logging before reconnection attempts
  • Enhanced error logging to include configuration ID for better identification
  • Added success logging when reconnection completes successfully

@bbrks bbrks merged commit fa7acc6 into main Jul 29, 2025
44 checks passed
@bbrks bbrks deleted the CBG-4757 branch July 29, 2025 11:44
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.

2 participants