brooktrails/forgejo-runner-operator
2
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

feat: add --debug-reconcile flag for reconciliation diagnostics #12

Merged
rcsheets merged 1 commit from feat/debug-reconcile into master 2026-03-26 05:28:25 +00:00
Conversation 1 Commits 1 Files changed 3 +58 -29
rcsheets commented 2026-03-26 05:26:11 +00:00
Owner
Copy link

When enabled, logs the full probability decision for every MaybeAudit
call: base probability, effective probability, random roll, whether
it triggered, reason for any adjustments, last audit time, and
tracked repo count. Also adds V(1) log at goroutine entry point.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

When enabled, logs the full probability decision for every MaybeAudit call: base probability, effective probability, random roll, whether it triggered, reason for any adjustments, last audit time, and tracked repo count. Also adds V(1) log at goroutine entry point. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
feat: add --debug-reconcile flag for reconciliation diagnostics
Some checks failed
CI / ci (pull_request) Has been cancelled
Details
E2E smoke test / e2e (pull_request) Has been cancelled
Details
CI / ci (push) Has been cancelled
Details
6bbd7373b3 
When enabled, logs the full probability decision for every MaybeAudit
call: base probability, effective probability, random roll, whether
it triggered, reason for any adjustments, last audit time, and
tracked repo count. Also adds V(1) log at goroutine entry point.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pr-reviewer-bot commented 2026-03-26 05:26:13 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.15.0 | Safety Check | anthropic | tracking id r-c4c374-0c409a
This is an AI-generated review and may contain mistakes.

Status: ✅ Completed

✅ Verdict: LGTM — This is a straightforward debugging feature addition with no obvious bugs or security issues.
Complexity: moderate

This PR adds a --debug-reconcile flag to enable detailed logging of reconciliation probability decisions. The changes are clean and focused:

  • Adds a boolean flag and passes it through configuration
  • Enhances logging to show probability calculations, random rolls, and decision reasons
  • Adds V(1) debug logging at goroutine entry point

No security issues or obvious bugs detected. The logic for probability adjustment and random roll evaluation remains unchanged, only logging is added.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.15.0 | Safety Check | anthropic | tracking id `r-c4c374-0c409a`* *This is an AI-generated review and may contain mistakes.* **Status:** ✅ Completed --- **✅ Verdict: LGTM** — This is a straightforward debugging feature addition with no obvious bugs or security issues. **Complexity:** moderate This PR adds a `--debug-reconcile` flag to enable detailed logging of reconciliation probability decisions. The changes are clean and focused: - Adds a boolean flag and passes it through configuration - Enhances logging to show probability calculations, random rolls, and decision reasons - Adds V(1) debug logging at goroutine entry point No security issues or obvious bugs detected. The logic for probability adjustment and random roll evaluation remains unchanged, only logging is added.
rcsheets merged commit fccc953b3d into master 2026-03-26 05:28:25 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
brooktrails/forgejo-runner-operator!12
No description provided.