11 Commits

Include renames

Author	SHA1	Message	Date
4 people	473f7529e1	Fix backup queue permanently stuck when a dequeued backup completes during the patch (#10521) ... * Fix backup queue permanently stuck when a dequeued backup completes during the patch Motivation: backupQueueReconciler patched a dequeued backup to ReadyToStart and only called backupTracker.AddReadyToStart on the next line. If backupReconciler picked up that patch and completed the backup (e.g. immediate FailedValidation while a BackupStorageLocation is briefly unavailable) before the queue controller reached that line, backupReconciler's Add + deferred Delete ran first, and the later AddReadyToStart re-inserted a tracker key nothing would ever delete again. backupTracker is in-memory and never reconciled against actual Backup phases, so RunningCount() stayed stuck at concurrentBackups and every later reconcile, including the periodic recheck, was refused at that gate -- the queue stopped dequeuing permanently until the deployment restarted. Approach: record the backup as ReadyToStart in the tracker before patching it, and roll that back if the patch itself fails, so the tracker entry always exists before the backup can become visible to any other reconciler. Also folds in two related fixes: the concurrency-refusal log line is now Info instead of Debug so a stuck queue is visible at the default log level, and the queue-position renumbering loop's error log (which built a logrus.Entry via log.WithError(errors.Wrapf(...)) but never called a terminal method on it, so it never actually logged anything) now emits properly. Validation: go build ./pkg/controller/..., go vet ./pkg/controller/..., and gofmt -l on both changed files are all clean. golangci-lint run ./pkg/controller/... reports no findings. go mod tidy produces a zero diff to go.mod/go.sum, matching this repo's verify-modules check. Mirrored this repo's own hack/test.sh invocation for this package (-short -vet=... -skip TestAPIs) and it passes; TestAPIs is a separate envtest suite that needs a local kubebuilder etcd binary not installed on this machine and fails identically on an unmodified checkout, so it is a pre-existing environment gap, not a regression. Added TestBackupQueueReconcilerTrackerNotLeakedWhenBackupCompletesDuringPatch, which uses a controller-runtime fake client with a Patch interceptor to simulate a racing reconciler completing the backup right after the ReadyToStart patch lands; it fails (RunningCount leaks to 1) against the pre-fix ordering and passes (RunningCount returns to 0) against the fix. Report: https://github.com/velero-io/velero/issues/10519 Signed-off-by: Pujitha Paladugu <10557236+pujitha24@users.noreply.github.com> Assisted-by: claude-sonnet-5 (via Claude Code) * Add changelog file for PR #10521 Signed-off-by: Pujitha Paladugu <10557236+pujitha24@users.noreply.github.com> * Add test covering tracker rollback when ReadyToStart patch fails Addresses review comment: verify backupTracker.RunningCount() returns to 0 when the ReadyToStart patch itself errors, covering the Delete rollback path alongside the existing race-condition regression test. Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> --------- Signed-off-by: Pujitha Paladugu <10557236+pujitha24@users.noreply.github.com> Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com> Co-authored-by: Pujitha Paladugu <10557236+pujitha24@users.noreply.github.com> Co-authored-by: Tiger Kaovilai <tkaovila@redhat.com>	2026-09-16 20:34:20 +08:00
Scott Seago	7e4797f588	Track running backup count via BackupTracker ... This avoids an unnecessary apiserver List call when the backup reconciler is already at capacity. Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 17:23:47 -05:00
Scott Seago	73864e31ff	Fix linters ... Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 17:04:55 -05:00
Scott Seago	8a95d512b3	make update, changelog ... Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 17:04:07 -05:00
Scott Seago	4d1802233a	add various scenarios to queue controller unit tests ... Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 17:01:09 -05:00
Scott Seagoandaider	845eee4e60	feat: Create backup queue controller and add to disableable list ... Co-authored-by: aider (gemini/gemini-2.5-pro) <aider@aider.chat> Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 16:46:56 -05:00
Scott Seago	e7b5d20f4c	Fix linters ... Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 16:39:10 -05:00
Scott Seago	aedc0fe5e2	make update, changelog ... Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 16:39:07 -05:00
Scott Seago	e0c08f03cf	add various scenarios to queue controller unit tests ... Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 16:38:41 -05:00
Scott Seagoandaider	d91d50f696	feat: Add concurrentBackups to backupQueueReconciler ... Co-authored-by: aider (gemini/gemini-2.5-pro) <aider@aider.chat> Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 16:28:08 -05:00
Scott Seagoandaider	5d02af3ce3	feat: Create backup queue controller and add to disableable list ... Co-authored-by: aider (gemini/gemini-2.5-pro) <aider@aider.chat> Signed-off-by: Scott Seago <sseago@redhat.com>	2025-12-02 16:28:08 -05:00