Skip to content

Conversation

@kramvan1
Copy link
Collaborator

@kramvan1 kramvan1 commented Jun 17, 2025

Includes the following changes

  • Rule lock retries
    • new Option RuleWatcherLockRetries
      • When the rule source is the Watcher, it will optionally retry getting the lock
      • RuleWatcherLockRetries(tries uint, wait time.Duration)
    • new lock label for watcher lock attempt metrics
  • Watcher log tweak
  • Lock session log tweak

ENSURE THE FOLLOWING ARE MET:

Below is a brief summary of PR requirements (full list).

  • Your PR title summarizes the changes at a high level (not simply "updated X" or "changes Y")
  • Your PR description links to the issue/design. If not available, includes a full description for the change.
  • Your code contains relevant unit tests.

By opening this PR for review, the author has agreed that these criteria must be met.

By approving this PR, the reviewers have also agreed these criteria have been met and it is ready to be merged.

@kramvan1 kramvan1 self-assigned this Jun 17, 2025
@kramvan1 kramvan1 marked this pull request as draft June 17, 2025 20:06
@kramvan1 kramvan1 force-pushed the watcher-event-wait branch 2 times, most recently from b9b59b5 to 9ae22c8 Compare June 17, 2025 20:48
@kramvan1 kramvan1 force-pushed the watcher-event-wait branch 2 times, most recently from ca377eb to 0210f8d Compare July 15, 2025 15:00
@kramvan1 kramvan1 force-pushed the watcher-event-wait branch 3 times, most recently from f315fd0 to 16c12a2 Compare October 21, 2025 16:43
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