Breadway/bos
1
0
Fork 0
Code Issues 5 Pull requests 1 Projects Releases 9 Packages Wiki Activity Actions

Snapshot history keeps only about 5 updates #56

Merged
Breadway merged 4 commits from fix/bos-27-snapshot-retention into main 2026-10-02 14:26:34 +08:00
Conversation 3 Commits 4 Files changed 5 +59 -3
Breadway commented 2026-09-30 01:30:33 +08:00
Owner
Copy link

Root Snapper retention keeps 2-20 normal and 4-10 important snapshots with quota-aware 20% free-space and 50% snapshot-space thresholds. The installer enables quota accounting, and the README and recovery docs explain the policy.

Validation: a headless KVM Arch guest with Btrfs root and 17 GiB free completed ten real pacman -S nano transactions. snap-pac created ten complete pre/post pairs, and all ten pairs survived number cleanup, including a repeat with NUMBER_MIN_AGE=0. Exact versions and reproduction commands are in docs/validation/bos-27-retention.md. Bash syntax and git diff --check pass. Earlier FAST_BUILD ISO build passed. Main was merged into this branch to resolve its changelog conflict.

Settings copy is tracked separately by BSET-20 and PR #22.

Closes #27

Root Snapper retention keeps 2-20 normal and 4-10 important snapshots with quota-aware 20% free-space and 50% snapshot-space thresholds. The installer enables quota accounting, and the README and recovery docs explain the policy. Validation: a headless KVM Arch guest with Btrfs root and 17 GiB free completed ten real pacman -S nano transactions. snap-pac created ten complete pre/post pairs, and all ten pairs survived number cleanup, including a repeat with NUMBER_MIN_AGE=0. Exact versions and reproduction commands are in docs/validation/bos-27-retention.md. Bash syntax and git diff --check pass. Earlier FAST_BUILD ISO build passed. Main was merged into this branch to resolve its changelog conflict. Settings copy is tracked separately by BSET-20 and PR #22. Closes #27
Breadway commented 2026-09-30 03:58:39 +08:00
Author
Owner
Copy link

This PR card tracks BOS-27 / PR #56. Ten pacman transaction/free-space acceptance is still missing; reviewer found the PR unmergeable. Parent BOS-27 is TODO. Synchronizing card status to TODO.

This PR card tracks BOS-27 / PR #56. Ten pacman transaction/free-space acceptance is still missing; reviewer found the PR unmergeable. Parent BOS-27 is TODO. Synchronizing card status to TODO.
Breadway commented 2026-09-30 11:02:01 +08:00
Author
Owner
Copy link

Independent review of PR #56: the change sets the requested Snapper retention values and documents them, but the acceptance criterion still requires ten real pacman -S transactions followed by confirming at least ten pre/post snapshot pairs while free space is available. I could not safely run that check in this environment. /dev/kvm is absent; this process has no elevated capabilities (NoNewPrivs and seccomp are active), and sudo cannot escalate here. A QEMU TCG guest is technically available, but there is no prepared installed guest, and the host filesystem is 95% full with 45 GB free while other BOS build work is present. Provisioning an installer guest for the real transaction test is not a safe validation path under those constraints. I did not run a simulated substitute or touch the live desktop, user configuration, or installed binaries. PR #56 is also currently reported unmergeable; Forgejo returns no CI statuses, consistent with this repo's lack of pull request CI. Returning BOS-27 and its PR card BOS-56 to TODO until the real acceptance can be demonstrated.

Independent review of PR #56: the change sets the requested Snapper retention values and documents them, but the acceptance criterion still requires ten real `pacman -S` transactions followed by confirming at least ten pre/post snapshot pairs while free space is available. I could not safely run that check in this environment. `/dev/kvm` is absent; this process has no elevated capabilities (`NoNewPrivs` and seccomp are active), and `sudo` cannot escalate here. A QEMU TCG guest is technically available, but there is no prepared installed guest, and the host filesystem is 95% full with 45 GB free while other BOS build work is present. Provisioning an installer guest for the real transaction test is not a safe validation path under those constraints. I did not run a simulated substitute or touch the live desktop, user configuration, or installed binaries. PR #56 is also currently reported unmergeable; Forgejo returns no CI statuses, consistent with this repo's lack of pull request CI. Returning BOS-27 and its PR card BOS-56 to TODO until the real acceptance can be demonstrated.
Merge main into snapshot retention rework
All checks were successful
Build and publish calamares / calamares (push) Successful in 3m4s
Details
Build and publish powerlevel10k / powerlevel10k (push) Successful in 2m12s
Details
Build and publish calamares / calamares (pull_request) Successful in 3m38s
Details
74b012f72b
Breadway commented 2026-09-30 23:55:44 +08:00
Author
Owner
Copy link

Independent review PASS at 44504c01af. Reviewed installer config/quota changes and independently read the preserved headless guest raw logs. They contain ten distinct pacman transaction markers, ten actual nano reinstalls, and ten complete snap-pac pre/post pairs. The separate age-eligible cleanup CSV retains all ten pairs after disabling the minimum-age protection. Final config matches NUMBER_LIMIT=2-20, NUMBER_LIMIT_IMPORTANT=4-10, FREE_LIMIT=0.2, SPACE_LIMIT=0.5 and QGROUP=1/0; guest has 17 GiB free. Bash syntax and diff check pass; no configured CI statuses. This meets the requested free-space transaction acceptance. It does not claim a fresh Calamares install or low-space stress test. Settings copy is tracked separately by BSET-20.

Independent review PASS at 44504c01af44149d90881c9f4b7ab78c010bf671. Reviewed installer config/quota changes and independently read the preserved headless guest raw logs. They contain ten distinct pacman transaction markers, ten actual nano reinstalls, and ten complete snap-pac pre/post pairs. The separate age-eligible cleanup CSV retains all ten pairs after disabling the minimum-age protection. Final config matches NUMBER_LIMIT=2-20, NUMBER_LIMIT_IMPORTANT=4-10, FREE_LIMIT=0.2, SPACE_LIMIT=0.5 and QGROUP=1/0; guest has 17 GiB free. Bash syntax and diff check pass; no configured CI statuses. This meets the requested free-space transaction acceptance. It does not claim a fresh Calamares install or low-space stress test. Settings copy is tracked separately by BSET-20.
# Conflicts:
#	CHANGELOG.md
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
P0

   
P1

   
P2

   
P3

   
audio

   
bakery

   
boot

   
breadbug

   
bug

   
build

   
ci

   
dependencies

   
design

   
disk

   
docs

   
encryption

   
final

   
firewall

   
graphics

   
hardware

   
idle

   
installer

   
keybinds

   
nvidia

   
packages

   
phase-1

   
polish

   
recovery

   
reliability

   
reproducibility

   
secure-boot

   
security

   
shell

   
signing

   
snapshots

   
supply-chain

   
swap

   
tech-debt

   
theming

   
ux

   
website
No labels
P0
P1
P2
P3
audio
bakery
boot
breadbug
bug
build
ci
dependencies
design
disk
docs
encryption
final
firewall
graphics
hardware
idle
installer
keybinds
nvidia
packages
phase-1
polish
recovery
reliability
reproducibility
secure-boot
security
shell
signing
snapshots
supply-chain
swap
tech-debt
theming
ux
website
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
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
Breadway/bos!56
No description provided.