Skip to content

fix(purefa_snap): Add error when state=copy and snapshot not found#979

Open
sdodsley wants to merge 2 commits intomasterfrom
fix/snap-copy-error-handling
Open

fix(purefa_snap): Add error when state=copy and snapshot not found#979
sdodsley wants to merge 2 commits intomasterfrom
fix/snap-copy-error-handling

Conversation

@sdodsley
Copy link
Member

@sdodsley sdodsley commented Mar 9, 2026

SUMMARY

Previously, when using state=copy with a snapshot that doesn't exist, the module would silently return changed=false without any error message. This made it difficult to diagnose why copies weren't being created.

Now the module fails with a clear error message indicating that the specified snapshot was not found.

Fixes #978

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

purefa_snap.py

Previously, when using state=copy with a snapshot that doesn't exist,
the module would silently return changed=false without any error message.
This made it difficult to diagnose why copies weren't being created.

Now the module fails with a clear error message indicating that the
specified snapshot was not found.

Fixes #978
@sdodsley sdodsley self-assigned this Mar 9, 2026
@sdodsley sdodsley added the bug Something isn't working label Mar 9, 2026
@sdodsley sdodsley marked this pull request as ready for review March 9, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

purefa_snap does not create a copy of snapshot to new volume

1 participant