linux-system-roles / snapshot

Ansible role to manage storage snapshots
MIT License
2 stars 5 forks source link

feat: add support for snapshot_lvm_vg_include #39

Closed richm closed 8 months ago

richm commented 8 months ago

Feature: Add parameter snapshot_lvm_vg_include which is a regular expression as used by Python re.search. When doing an operation with all volume groups (snapshot_lvm_all_vgs: true), only use volume groups whose names match the given pattern.

Reason: There may be cases where you want to use all instead of providing a single volume group or a snapset, but do not want the operation to apply to all volume groups on the system.

Result: Users can manage snapshots of a subset of all volume groups.

This is being used for test code because our test machines use volume groups that we do not want snapshot operations to apply to.

Also fix a test error where the script output goes to stderr rather than stdout, so we need to look for the error message in both.

Signed-off-by: Rich Megginson rmeggins@redhat.com

richm commented 8 months ago

[citest]

richm commented 8 months ago

[citest]