Thomas Huth <thuth@xxxxxxxxxx> writes: > According to https://marc.info/?l=fedora-devel-list&m=171934833215726 > the GlusterFS development effectively ended. Thus mark it as deprecated > in QEMU, so we can remove it in a future release if the project does > not gain momentum again. > > Acked-by: Niels de Vos <ndevos@xxxxxxxxxx> > Signed-off-by: Thomas Huth <thuth@xxxxxxxxxx> > --- > v2: Mark it as deprecated in the QAPI and print a warning once, too > > docs/about/deprecated.rst | 9 +++++++++ > qapi/block-core.json | 7 ++++++- > block/gluster.c | 2 ++ > 3 files changed, 17 insertions(+), 1 deletion(-) > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst > index ed31d4b0b2..b231aa3948 100644 > --- a/docs/about/deprecated.rst > +++ b/docs/about/deprecated.rst > @@ -395,6 +395,15 @@ Specifying the iSCSI password in plain text on the command line using the > used instead, to refer to a ``--object secret...`` instance that provides > a password via a file, or encrypted. > > +``gluster`` backend (since 9.2) > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +According to https://marc.info/?l=fedora-devel-list&m=171934833215726 > +the GlusterFS development effectively ended. Unless the development > +gains momentum again, the QEMU project might remove the gluster backend > +in a future release. > + > + > Character device options > '''''''''''''''''''''''' > > diff --git a/qapi/block-core.json b/qapi/block-core.json > index 9f6dd59298..cb7cb1c0ed 100644 > --- a/qapi/block-core.json > +++ b/qapi/block-core.json > @@ -3187,12 +3187,17 @@ > # > # @snapshot-access: Since 7.0 > # > +# Features: > +# > +# @deprecated: Member @gluster is deprecated since GlusterFS ceased development End the sentence with full stop, and wrap the line. Suggest: # @deprecated: Member @gluster is deprecated because GlusterFS # development ceased. > +# > # Since: 2.9 > ## > { 'enum': 'BlockdevDriver', > 'data': [ 'blkdebug', 'blklogwrites', 'blkreplay', 'blkverify', 'bochs', > 'cloop', 'compress', 'copy-before-write', 'copy-on-read', 'dmg', > - 'file', 'snapshot-access', 'ftp', 'ftps', 'gluster', > + 'file', 'snapshot-access', 'ftp', 'ftps', > + {'name': 'gluster', 'features': [ 'deprecated' ] }, > {'name': 'host_cdrom', 'if': 'HAVE_HOST_BLOCK_DEVICE' }, > {'name': 'host_device', 'if': 'HAVE_HOST_BLOCK_DEVICE' }, > 'http', 'https', I wonder why it's not 'if': 'CONFIG_GLUSTERFS'. Probably not worth exploring now. With the doc comment tidied up: Acked-by: Markus Armbruster <armbru@xxxxxxxxxx> [...]
