This repository has been archived by the owner on Jan 15, 2021. It is now read-only.
support checking the snapshot age #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
otherwise we would also match thin-provisioned volumes example lvs output: % sudo lvs LV VG Attr LSize Pool Origin Data% Meta% Move Log Cpy%Sync Convert home fedora -wi-ao---- 20.00g root fedora owi-aos--- 20.00g root-snap fedora swi-a-s--- 1.00g root 0.35 swap fedora -wi-ao---- 5.81g vm-osp7compute-root fedora Vwi-a-tz-- 10.00g vmthinpool 0.00 vm-osp7controller-root fedora Vwi-a-tz-- 10.00g vmthinpool 100.00 vm-osp7dir-root fedora Vwi-a-tz-- 10.00g vmthinpool 100.00 vm-rhel66-root fedora Vwi-a-tz-- 10.00g vmthinpool 9.30 vm-rhel71-root fedora Vwi-a-tz-- 10.00g vmthinpool 9.58 vm-rhevh35-root fedora Vwi-a-tz-- 10.00g vmthinpool 8.27 vm-rhevm35-root fedora Vwi-a-tz-- 10.00g vmthinpool 54.21 vm-sat61-pulp fedora Vwi-aotz-- 40.00g vmthinpool 67.43 vm-sat61-root fedora Vwi-aotz-- 15.00g vmthinpool 50.99 vm-sat61-swap fedora Vwi-aotz-- 4.00g vmthinpool 9.21 vmthinpool fedora twi-aotz-- 70.00g 90.17 78.80
that makes parsing the output easier and stable for future changes
|
Thanks for this. However, I noticed that |
|
Hi, meh, I only checked on RHEL6 and expected it to work on others too. Will have look and come back to you :) Greets Edit: Debian Wheezy (lvm2 2.02.95-8) does not support this either. Debian Jessie (lvm2 2.02.111-2.2) supports this just fine. Ubuntu version mentioned above is from Precise, 12.04 LTS. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
we have a need to verify that we do not have too old snapshots (people keep forgetting things, you know). To fix that, I have implemented an optional age check (in days).
Code is dependent on the new lvs call from #4, sorry :)
Greets