Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert - wait for all snapshots to drain before returning #15

Merged
merged 2 commits into from
Oct 2, 2023

Conversation

ygalblum
Copy link
Collaborator

Add a check that the all snapshots were drained
Add test
Add changelog fragement

Add a check that the all snapshots were drained
Add test
Add changelog fragement

Signed-off-by: Ygal Blum <[email protected]>
@ygalblum ygalblum force-pushed the revert-wait-for-drain branch from ac9257d to e632f2a Compare September 28, 2023 09:00
Copy link
Member

@swapdisk swapdisk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made one suggestion. Otherwise, LGTM.

roles/lvm_snapshots/tasks/revert.yml Outdated Show resolved Hide resolved
Add group name to the select filter to make sure the returned list has only one result

Co-authored-by: Bob Mader <[email protected]>
@ygalblum
Copy link
Collaborator Author

ygalblum commented Oct 1, 2023

I made one suggestion. Otherwise, LGTM.

I've added your fix (thanks for that).

@swapdisk swapdisk merged commit fbecaf9 into redhat-cop:main Oct 2, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants