[libvirt] [PATCH 0/1] Add shrink flag to blockresize command

martinsson.patrik@gmail.com posted 1 patch 4 years, 5 months ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20191005185621.1517545-1-martinsson.patrik@gmail.com
src/qemu/qemu_driver.c | 15 ++++++++++++++-
tools/virsh-domain.c   |  7 +++++++
tools/virsh.pod        |  9 ++++++++-
3 files changed, 29 insertions(+), 2 deletions(-)
[libvirt] [PATCH 0/1] Add shrink flag to blockresize command
Posted by martinsson.patrik@gmail.com 4 years, 5 months ago
From: patchon <martinsson.patrik@gmail.com>

This changes the behaviour when shrinking a block device. This should be
a fairly uncommon thing to do (I think), so I don't think one have to
worry about breaking anything when introducing the new flag (--shrink).

This also mimics the existing behaviour of the vol-resize command, which
I personally think makes sense.

Unfortunately this check was not present when I, one late Friday
afternoon, was asked to extend a production-mysql-slave-device. This
made me accidentally shrink a 2.5TB device with about 150GB. Needless
to say - that Friday afternoon did not end with the obligatory
weekend-beer-celebration.

Non the less, I think this makes sense from an end user perspective.

Also, there is actually one mention of this on behaviour already at
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=902171

patchon (1):
  Add shrink flag to blockresize command

 src/qemu/qemu_driver.c | 15 ++++++++++++++-
 tools/virsh-domain.c   |  7 +++++++
 tools/virsh.pod        |  9 ++++++++-
 3 files changed, 29 insertions(+), 2 deletions(-)

-- 
2.23.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list