midwitmoneymgmt/m3-infra
2
1
Fork 0
Code Issues Pull requests Projects Releases Activity Actions

Fix local pool import: match actual ZFS config #43

Merged
AlexCaswen merged 3 commits from 46-import-local-pool into main 2026-05-09 17:25:22 +00:00
Conversation 6 Commits 3 Files changed 1 +7 -2
AlexCaswen commented 2026-05-09 17:23:07 +00:00 (Migrated from gitlab.com)
Copy link

What

Fixes the failed import from !42 — the incus_storage_pool.local resource was declared without source, zfs.pool_name, or description, so OpenTofu saw a config mismatch and tried to destroy/recreate the pool (which failed because it's in use).

Fix

Added the actual pool config to match incus storage show local:

  • source = "local/incus"
  • zfs.pool_name = "local/incus"
  • description = "Local storage pool (on system drive)"

Expected behavior

The import block re-imports (or recognizes the pool is already in state), sees the config now matches, and produces no changes.

Ref #46

## What Fixes the failed import from !42 — the `incus_storage_pool.local` resource was declared without `source`, `zfs.pool_name`, or `description`, so OpenTofu saw a config mismatch and tried to destroy/recreate the pool (which failed because it's in use). ## Fix Added the actual pool config to match `incus storage show local`: - `source = "local/incus"` - `zfs.pool_name = "local/incus"` - `description = "Local storage pool (on system drive)"` ## Expected behavior The import block re-imports (or recognizes the pool is already in state), sees the config now matches, and produces no changes. Ref #46
AlexCaswen (Migrated from gitlab.com) approved these changes 2026-05-09 17:23:07 +00:00
AlexCaswen commented 2026-05-09 17:23:31 +00:00 (Migrated from gitlab.com)
Copy link

assigned to @AlexCaswen

assigned to @AlexCaswen
AlexCaswen commented 2026-05-09 17:24:20 +00:00 (Migrated from gitlab.com)
Copy link

added 3 commits

  • cdc0190c...374463c9 - 2 commits from branch main
  • dfafe826 - Merge branch 'main' into '46-import-local-pool'

Compare with previous version

added 3 commits <ul><li>cdc0190c...374463c9 - 2 commits from branch <code>main</code></li><li>dfafe826 - Merge branch &#39;main&#39; into &#39;46-import-local-pool&#39;</li></ul> [Compare with previous version](/AlexCaswen/m3-infra/-/merge_requests/43/diffs?diff_id=1791404711&start_sha=cdc0190c7a6271ff78561e81d5534be498750a81)
AlexCaswen commented 2026-05-09 17:25:00 +00:00 (Migrated from gitlab.com)
Copy link

approved this merge request

approved this merge request
AlexCaswen commented 2026-05-09 17:25:22 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 0398d05e64

mentioned in commit 0398d05e64d2e918839b204feca79a9b35e6b0e2
AlexCaswen (Migrated from gitlab.com) merged commit 0398d05e64 into main 2026-05-09 17:25:22 +00:00
AlexCaswen commented 2026-05-09 17:31:16 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in merge request !44

mentioned in merge request !44
Sign in to join this conversation.
Reviewers
No reviewers
AlexCaswen
Labels
Clear labels   
IaC

   
admin

   
agent-containers

   
automation

   
blockchain

   
cleanup

   
data

   
documentation

   
hardware

   
hotfix

   
infrastructure

   
maintenance

   
metrics

   
ml

   
monitoring

   
networking

   
resilience

   
security

   
tailscale

   
trading
No labels
IaC
admin
agent-containers
automation
blockchain
cleanup
data
documentation
hardware
hotfix
infrastructure
maintenance
metrics
ml
monitoring
networking
resilience
security
tailscale
trading
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
midwitmoneymgmt/m3-infra!43
No description provided.