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

Improve qvm-volume tool #60

Merged
merged 5 commits into from
Mar 21, 2018
Merged

Conversation

marmarek
Copy link
Member

The final part for volume snapshots handling, especially revisions_to_keep
setting. This also adds an option to get more information about a single volume
(size, usage, available revisions etc).

Fixes QubesOS/qubes-issues#3256

@codecov
Copy link

codecov bot commented Mar 19, 2018

Codecov Report

Merging #60 into master will increase coverage by 0.28%.
The diff coverage is 80.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   74.92%   75.21%   +0.28%     
==========================================
  Files          47       47              
  Lines        5269     5338      +69     
==========================================
+ Hits         3948     4015      +67     
- Misses       1321     1323       +2
Impacted Files Coverage Δ
qubesadmin/storage.py 80.92% <42.1%> (-2.79%) ⬇️
qubesadmin/tools/qvm_volume.py 93.29% <94.23%> (+0.43%) ⬆️
qubesadmin/tools/__init__.py 73.5% <0%> (-0.38%) ⬇️
qubesadmin/tools/qvm_run.py 83.46% <0%> (+3.93%) ⬆️
qubesadmin/spinner.py 77.41% <0%> (+6.45%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9b9eb1...70b15c2. Read the comment docs.

marmarta added a commit to marmarta/qubes-desktop-linux-manager that referenced this pull request Mar 19, 2018
Added an auto-starting disk space widget. It warns the user if there's
less than 5% free in any pool. (Further improvements, like configurable
warning threshold, will come).

Depends on QubesOS/qubes-core-admin-client#60
marmarta added a commit to marmarta/qubes-desktop-linux-manager that referenced this pull request Mar 19, 2018
Added an auto-starting disk space widget. It warns the user if there's
less than 5% free in any pool. (Further improvements, like configurable
warning threshold, will come).

Depends on QubesOS/qubes-core-admin-client#60
marmarta added a commit to marmarta/qubes-desktop-linux-manager that referenced this pull request Mar 19, 2018
Added an auto-starting disk space widget. It warns the user if there's
less than 5% free in any pool. (Further improvements, like configurable
warning threshold, will come).

Depends on QubesOS/qubes-core-admin-client#60
marmarta added a commit to marmarta/qubes-desktop-linux-manager that referenced this pull request Mar 19, 2018
Added an auto-starting disk space widget. It warns the user if there's
less than 5% free in any pool. (Further improvements, like configurable
warning threshold, will come).

Depends on QubesOS/qubes-core-admin-client#60
marmarta added a commit to marmarta/qubes-desktop-linux-manager that referenced this pull request Mar 19, 2018
Added an auto-starting disk space widget. It warns the user if there's
less than 5% free in any pool. (Further improvements, like configurable
warning threshold, will come).

Depends on QubesOS/qubes-core-admin-client#60
fixes QubesOS/qubes-issues#3240
references QubesOS/qubes-issues#3241
And return None (instead of raising KeyError) when pool driver
does not provide such information.
Use *VMNAME:VOLUME* instead of *POOL_NAME:VOLUME_ID*, because this is
the default form.
@marmarek marmarek force-pushed the qvm-volume-actions branch 2 times, most recently from 1e72d33 to 46a8bb9 Compare March 20, 2018 16:52
Right now Admin API backend will refuse to shrink volume anyway, but
we're planning to relax this restriction. Make sure the client side
(qvm-volume tool here, GUI VM settings already have this in place) will
employ appropriate safety check.

QubesOS/qubes-issues#3725
@marmarek marmarek force-pushed the qvm-volume-actions branch from 46a8bb9 to 70b15c2 Compare March 20, 2018 16:53
@marmarek marmarek merged commit 70b15c2 into QubesOS:master Mar 21, 2018
@marmarek marmarek deleted the qvm-volume-actions branch July 16, 2018 20:10
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.

1 participant