Re: [libvirt PATCH 2/2] remove sheepdog support from libvirt

On Mon, Apr 19, 2021 at 12:47:59 +0200, Pavel Hrdina wrote: > Upstream sheepdog project doesn't have any active development for almost > 3 years as the last commit is from 28/06/2018 [1]. > > Looking into repology [2] the latest version 1.0.1 is included only in > Fedora (since Fedora 26). There are only few other distributions having > older sheepdog versions: Debian oldstable (stretch), EPEL 6, Ubuntu > 18.04 and older and few other less-common distributions. > > Based on the data above drop support for sheepdog from libvirt. > > [1] <https://github.com/sheepdog/sheepdog/commits/master> > [2] <https://repology.org/project/sheepdog/versions> > > Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com&gt; > --- > docs/formatdomain.rst | 13 +- > docs/formatstorage.html.in | 5 +- > docs/kbase/rpm-deployment.rst | 6 - > docs/manpages/virsh.rst | 7 +- > docs/schemas/domaincommon.rng | 1 - > docs/schemas/storagepool.rng | 22 - > docs/storage.html.in | 59 --- > libvirt.spec.in | 36 -- > meson.build | 6 - > meson_options.txt | 1 - > mingw-libvirt.spec.in | 1 - > po/POTFILES.in | 1 - > src/conf/domain_conf.c | 1 - > src/conf/storage_conf.c | 12 +- > src/conf/storage_conf.h | 2 - > src/conf/storage_source_conf.c | 4 - > src/conf/storage_source_conf.h | 1 - > src/conf/virstorageobj.c | 9 - > src/libvirt-storage.c | 1 - > src/libxl/libxl_conf.c | 1 - > src/libxl/xen_xl.c | 1 - > src/qemu/qemu_block.c | 41 -- > src/qemu/qemu_command.c | 20 - > src/qemu/qemu_domain.c | 1 - > src/qemu/qemu_snapshot.c | 3 - > src/storage/meson.build | 25 -- > src/storage/storage_backend.c | 6 - > src/storage/storage_backend_sheepdog.c | 378 ------------------ > src/storage/storage_backend_sheepdog.h | 25 -- > src/storage/storage_backend_sheepdog_priv.h | 30 -- > src/storage/storage_driver.c | 1 - > .../storage_source_backingstore.c | 44 -- > src/test/test_driver.c | 1 - > tests/meson.build | 6 - > tests/qemublocktest.c | 4 - > .../imagecreate/network-sheepdog-qcow2.json | 20 - > .../imagecreate/network-sheepdog-qcow2.xml | 12 - > tests/qemusecuritytest.c | 1 - > .../disk-network-sheepdog.args | 30 -- > .../disk-network-sheepdog.x86_64-2.12.0.args | 34 -- > .../disk-network-sheepdog.x86_64-latest.args | 38 -- > .../disk-network-sheepdog.xml | 37 -- > tests/qemuxml2argvtest.c | 3 - > .../disk-network-sheepdog.xml | 43 -- > tests/qemuxml2xmltest.c | 1 - > tests/storagebackendsheepdogtest.c | 206 ---------- > .../storagepoolcapsschemadata/poolcaps-fs.xml | 2 - > .../poolcaps-full.xml | 2 - > tests/storagepoolxml2argvtest.c | 2 - > tests/storagepoolxml2xmlin/pool-sheepdog.xml | 8 - > tests/storagepoolxml2xmlout/pool-sheepdog.xml | 11 - > tests/storagepoolxml2xmltest.c | 1 - > tests/storagevolxml2xmlin/vol-sheepdog.xml | 10 - > tests/storagevolxml2xmlout/vol-sheepdog.xml | 8 - > tests/storagevolxml2xmltest.c | 1 - > .../deflatten-qemu-sheepdog-in.json | 11 - > .../deflatten-qemu-sheepdog-out.json | 13 - > tests/virjsontest.c | 1 - > tests/virstoragetest.c | 21 - > tools/virsh-pool.c | 3 - > tools/virsh.c | 3 - > 61 files changed, 8 insertions(+), 1289 deletions(-) > delete mode 100644 src/storage/storage_backend_sheepdog.c > delete mode 100644 src/storage/storage_backend_sheepdog.h > delete mode 100644 src/storage/storage_backend_sheepdog_priv.h > delete mode 100644 tests/qemublocktestdata/imagecreate/network-sheepdog-qcow2.json > delete mode 100644 tests/qemublocktestdata/imagecreate/network-sheepdog-qcow2.xml > delete mode 100644 tests/qemuxml2argvdata/disk-network-sheepdog.args > delete mode 100644 tests/qemuxml2argvdata/disk-network-sheepdog.x86_64-2.12.0.args > delete mode 100644 tests/qemuxml2argvdata/disk-network-sheepdog.x86_64-latest.args > delete mode 100644 tests/qemuxml2argvdata/disk-network-sheepdog.xml > delete mode 100644 tests/qemuxml2xmloutdata/disk-network-sheepdog.xml > delete mode 100644 tests/storagebackendsheepdogtest.c > delete mode 100644 tests/storagepoolxml2xmlin/pool-sheepdog.xml > delete mode 100644 tests/storagepoolxml2xmlout/pool-sheepdog.xml > delete mode 100644 tests/storagevolxml2xmlin/vol-sheepdog.xml > delete mode 100644 tests/storagevolxml2xmlout/vol-sheepdog.xml > delete mode 100644 tests/virjsondata/deflatten-qemu-sheepdog-in.json > delete mode 100644 tests/virjsondata/deflatten-qemu-sheepdog-out.json While I don't have any problems with removing the sheepdog storage driver I don't think we should remove both the storage driver and qemu driver interactions in one commit and at the same time I also don't

think we should remove the support for launching QEMU guests with sheepdog backing as that has negligible maint burden.