Fix backup capture, mounts, and run history #5

Merged
kleb merged 10 commits from fix/windows-backup-reliability into main 2026-07-30 06:12:09 +02:00
Owner

Summary

  • make filesystem capture resilient to inaccessible Windows alternate data streams while preserving supported metadata and links
  • persist completed job progress and protocols across daemon restarts, and report maintenance operations with their concrete job kinds
  • validate Windows mount targets, avoid drive letters used by interactive sessions, and expose read-only WinFsp mounts to unelevated users
  • correct repository and mount UI states and include the related generated API updates

Validation

  • go test ./...
  • go test -race -count=1 ./internal/mount/winfsp ./internal/service
  • go vet ./...
  • KBACKUP_HOST_MUTATING_TESTS=1 go test -tags=integration -run '^TestMount_FileTree_RealDriverMount$' -v ./internal/mount/winfsp -count=1
  • web unit tests (44 tests)
  • production web build and installer verification
## Summary - make filesystem capture resilient to inaccessible Windows alternate data streams while preserving supported metadata and links - persist completed job progress and protocols across daemon restarts, and report maintenance operations with their concrete job kinds - validate Windows mount targets, avoid drive letters used by interactive sessions, and expose read-only WinFsp mounts to unelevated users - correct repository and mount UI states and include the related generated API updates ## Validation - `go test ./...` - `go test -race -count=1 ./internal/mount/winfsp ./internal/service` - `go vet ./...` - `KBACKUP_HOST_MUTATING_TESTS=1 go test -tags=integration -run '^TestMount_FileTree_RealDriverMount$' -v ./internal/mount/winfsp -count=1` - web unit tests (44 tests) - production web build and installer verification
Collaborator

kReview review

Verdict: no findings

No findings to address in the reviewed diff.

Overall risk is low; no concrete correctness, security, or data-integrity defect is provable from the supplied diff.

Excluded as generated or vendored (not reviewed):

6 excluded files
  • build/ci/README.md
  • build/qualification/README.md
  • build/qualification/Run-Windows.ps1
  • build/qualification/run-debian.sh
  • build/windows/Build-Installer.ps1
  • go.sum

Reviewed by kReview at 53fc8e49bd. This comment is conservative and based only on the PR diff and metadata.

Est. cost ~$7.36 total (1.3M in / 32.9k out) · this run ~$0.88 (154.4k in / 4.3k out) / gpt-5.6-sol.

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:53fc8e49bd6256c0286e78daf29ab8725241256b --> ## kReview review **Verdict:** no findings No findings to address in the reviewed diff. Overall risk is low; no concrete correctness, security, or data-integrity defect is provable from the supplied diff. _Excluded as generated or vendored (not reviewed):_ <details> <summary>6 excluded files</summary> - `build/ci/README.md` - `build/qualification/README.md` - `build/qualification/Run-Windows.ps1` - `build/qualification/run-debian.sh` - `build/windows/Build-Installer.ps1` - `go.sum` </details> _Reviewed by kReview at `53fc8e49bd`. This comment is conservative and based only on the PR diff and metadata._ _Est. cost ~$7.36 total (1.3M in / 32.9k out) · this run ~$0.88 (154.4k in / 4.3k out) / gpt-5.6-sol._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiNTNmYzhlNDliZDYyNTZjMDI4NmU3OGRhZjI5YWI4NzI1MjQxMjU2YiIsInN1bW1hcnkiOiJPdmVyYWxsIHJpc2sgaXMgbG93OyBubyBjb25jcmV0ZSBjb3JyZWN0bmVzcywgc2VjdXJpdHksIG9yIGRhdGEtaW50ZWdyaXR5IGRlZmVjdCBpcyBwcm92YWJsZSBmcm9tIHRoZSBzdXBwbGllZCBkaWZmLiIsImZpbmRpbmdzIjpbXSwiY3VtdWxhdGl2ZUNvc3QiOnsidXNkIjo3LjM1OTc5OTk5OTk5OTk5OSwiaW5wdXRUb2tlbnMiOjEyOTE0ODAsIm91dHB1dFRva2VucyI6MzI4NjQsIm1vZGVsIjoiZ3B0LTUuNi1zb2wifX0= -->
kleb merged commit aca0231f68 into main 2026-07-30 06:12:09 +02:00
kleb deleted branch fix/windows-backup-reliability 2026-07-30 06:12:09 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
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
kleb/kBackup!5
No description provided.