feat(zfs): manage jeeves datasets declaratively from nix #47

Open

Richie wants to merge 1 commits from feature/manage-jeeves-datasets-declaratively-from-nix into main

pull from: feature/manage-jeeves-datasets-declaratively-from-nix

merge into: :main

:main

:automation/update-flake-lock

:feature/manage-jeeves-datasets-declaratively-from-nix

:feature/Implement-authentication-system-with-WebAuthn-and-password-fallback

:feature/adding-tortoise

:bugfix/fixing-update-flake-lock.yaml-permissions

:feature/adding-qwen3.6-27b

:update_flake_lock_action

:feature/removing-brain_substituter.nix-from-bob

:feature/creating-prompt_bench

:claude/add-line-counter-tool-zaW28

:feature/setting-up-bluesky-firehose

:claude/sheet-music-ocr-mxml-4OGMU

:feature/adding-congress-data-to-database

:claude/improve-code-coverage-aW1aD

:claude/implement-todo-item-w2KGe

:feature/adding-open_webui-to-jeeves

:feature/testing-unshare

:bugfix/fixed-energy-out-integrations

:feature/adding-fix_eval_warnings.yml

:feature/building-safe_reboot

:feature/fixing-jeeves-locked-linux-firmware-and-6.12.52

:feature/adding-cool-tools

:feature/adding-xf86_input_wacom

:feature/adding-making-esphome-battery-more-stable

:feature/linux-6.17

:bugfix/added-math-the-jeeves

:feature/improving-treefmt-action-debugging

:feature/added-merge_group

:feature/testing-merge_group

:feature/removing-auto-update

:feature/linux-6.15

:feature/setting-up-unifi-video-archiving

:feature/updating-system_tools

:feature/decreasing-cloud-flair-usage

:feature/creating-a-custom-installer-iso

:feature/setting-up-mxr-deploment

:feature/adding-system-for-brendan

:feature/testing-microvm

:feature/testing-cosmic

:feature/building-nixos-based-router

:feature/setting-up-transmission

:feature/improving-muninns-bluetooth

:intresting-error

:feature/setting-up-keyFile-for-muninn

:feature/setting-up-cleves-for-muninn

:feature/testing-tang-and-clevis

:feature/testing-nixos-router

:updating-bobs-zfs-settings

Conversation 0 Commits 1 Files Changed 11

+1169 -184

Richie commented 2026-07-27 23:38:05 -04:00

Owner

Copy Link

Copy Source

Add a zfs_manager module that reconciles the live datasets on jeeves
against a nix declaration, and generate the snapshot retention config
from that same declaration so the two can no longer drift apart.

systems/jeeves/datasets.nix declares every dataset on the media, storage
and scratch pools, nested the way zfs nests them and flattened into
pool/parent/child names. Values were transcribed from the live pools
rather than from scripts/zfs.sh, which had gone stale: acltype reads back
as posix, and media/secure/important, scratch/kestra and storage/nomad
were never recorded. root_pool datasets are declared for retention only,
their properties stay unmanaged for now.

python.tools.zfs_manager creates missing datasets and corrects drifted
properties, and never destroys anything. Undeclared properties are judged
by the zfs source field, so inherited and default values stay quiet while
locally set ones warn. Size values are normalised to bytes so that 16K and
16384 do not re-issue zfs set on every run.

vars.nix now derives its paths from the declared mountpoints instead of
repeating them, dropping three zfs_* keys that nothing referenced.

Replaces systems/jeeves/snapshot_config.toml, which listed a dataset that
does not exist and omitted thirteen that do.

Add a zfs_manager module that reconciles the live datasets on jeeves against a nix declaration, and generate the snapshot retention config from that same declaration so the two can no longer drift apart. systems/jeeves/datasets.nix declares every dataset on the media, storage and scratch pools, nested the way zfs nests them and flattened into pool/parent/child names. Values were transcribed from the live pools rather than from scripts/zfs.sh, which had gone stale: acltype reads back as posix, and media/secure/important, scratch/kestra and storage/nomad were never recorded. root_pool datasets are declared for retention only, their properties stay unmanaged for now. python.tools.zfs_manager creates missing datasets and corrects drifted properties, and never destroys anything. Undeclared properties are judged by the zfs source field, so inherited and default values stay quiet while locally set ones warn. Size values are normalised to bytes so that 16K and 16384 do not re-issue zfs set on every run. vars.nix now derives its paths from the declared mountpoints instead of repeating them, dropping three zfs_* keys that nothing referenced. Replaces systems/jeeves/snapshot_config.toml, which listed a dataset that does not exist and omitted thirteen that do.

Richie added 1 commit 2026-07-27 23:38:06 -04:00

feat(zfs): manage jeeves datasets declaratively from nix ... ae18feb0fd

Add a zfs_manager module that reconciles the live datasets on jeeves
against a nix declaration, and generate the snapshot retention config
from that same declaration so the two can no longer drift apart.

systems/jeeves/datasets.nix declares every dataset on the media, storage
and scratch pools, nested the way zfs nests them and flattened into
pool/parent/child names. Values were transcribed from the live pools
rather than from scripts/zfs.sh, which had gone stale: acltype reads back
as posix, and media/secure/important, scratch/kestra and storage/nomad
were never recorded. root_pool datasets are declared for retention only,
their properties stay unmanaged for now.

python.tools.zfs_manager creates missing datasets and corrects drifted
properties, and never destroys anything. Undeclared properties are judged
by the zfs source field, so inherited and default values stay quiet while
locally set ones warn. Size values are normalised to bytes so that 16K and
16384 do not re-issue zfs set on every run.

vars.nix now derives its paths from the declared mountpoints instead of
repeating them, dropping three zfs_* keys that nothing referenced.

Replaces systems/jeeves/snapshot_config.toml, which listed a dataset that
does not exist and omitted thirteen that do.

All checks were successful

Hide all checks

treefmt / nix fmt (pull_request) Successful in 6s

Details

pytest / pytest (pull_request) Successful in 37s

Details

test ebook search / test-ebook-search (pull_request) Successful in 44s

Details

build_systems / build-brain (pull_request) Successful in 57s

Details

build_systems / build-bob (pull_request) Successful in 58s

Details

build_systems / build-rhapsody-in-green (pull_request) Successful in 1m11s

Details

build_systems / build-jeeves (pull_request) Successful in 3m1s

Details

Checking for merge conflicts…

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feature/manage-jeeves-datasets-declaratively-from-nix:feature/manage-jeeves-datasets-declaratively-from-nix

git checkout feature/manage-jeeves-datasets-declaratively-from-nix

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

automated

bug

Something is not working

dependencies

duplicate

This issue or pull request already exists

enhancement

New feature

flake_lock_update

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

Richie

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Richie/dotfiles#47