[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] qemu-img: Let "info" warn and go ahead with
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U |
Date: |
Mon, 8 Jan 2018 15:41:36 +0100 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
Am 05.01.2018 um 07:55 hat Fam Zheng geschrieben:
> Management and users are accustomed to "qemu-img info" to query status of
> images even when they are used by guests. Since image locking was added, the
> -U
> (--force-share) option is needed for that to work. The reason has been that
> due
> to possible race with image header update, the output can be misleading.
>
> But what are likely to happen after we emit the error are that, for
> interactive
> users, '-U' will be used and the command retried; for management (nova, RHV,
> etc.), the operation is broken with no knob to workaround this.
>
> This series changes that error to a warning so that it doesn't get in the way.
Are management tools actually doing this? There is no good reason to
call 'qemu-img info' for an image that is in use by a VM.
If no, NACK. Automatically disabling locking because it can be
inconvenient defeats the purpose of locking.
If yes, clearly indicate that this usage is deprecated and we'll turn
this into an error again with 2.13. Then management tools can be fixed
in time.
Kevin
- [Qemu-devel] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Fam Zheng, 2018/01/05
- [Qemu-devel] [PATCH 2/2] qemu-img: info: try -U automatically, Fam Zheng, 2018/01/05
- [Qemu-devel] [PATCH 1/2] qemu-img: Move img_open error reporting to callers, Fam Zheng, 2018/01/05
- Re: [Qemu-devel] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U,
Kevin Wolf <=
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Nir Soffer, 2018/01/08
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Kevin Wolf, 2018/01/08
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Fam Zheng, 2018/01/09
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Kevin Wolf, 2018/01/09
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Ala Hino, 2018/01/09
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Eric Blake, 2018/01/09
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Ala Hino, 2018/01/09
- Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Daniel P. Berrange, 2018/01/10
Re: [Qemu-devel] [PATCH 0/2] qemu-img: Let "info" warn and go ahead without -U, Daniel P. Berrange, 2018/01/10