[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/2] qapi/block-core: Overlays are not snapsh
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] qapi/block-core: Overlays are not snapshots |
Date: |
Wed, 05 Jun 2019 07:43:35 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Max Reitz <address@hidden> writes:
> A snapshot is something that reflects the state of something at a
> certain point in time. It does not change.
>
> The file our snapshot commands create (or the node they install) is not
> a snapshot, as it does change over time. It is an overlay. We cannot
> do anything about the parameter names,
We certainly could: add new parameter, default to the old one, error out
when both are given, deprecate the old one. Way more trouble than it's
worth.
If the QAPI machinery made such renames as simple as
'overlay-file': {'type': 'str', 'alias': 'snapshot-file'}
we could consider it. Of course, whether enhancing the machinery that
way would be worthwhile depends on complexity and on use. I think we
got bigger fish to fry.
> but we can at least adjust the
> descriptions to reflect that fact.
>
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
Since you already got competent review, I merely glanced at the patches,
and didn't check for completeness. No objections.
- [Qemu-devel] [PATCH v2 0/2] blockdev: Overlays are not snapshots, Max Reitz, 2019/06/03
- [Qemu-devel] [PATCH v2 2/2] blockdev: Overlays are not snapshots, Max Reitz, 2019/06/03
- [Qemu-devel] [PATCH v2 1/2] qapi/block-core: Overlays are not snapshots, Max Reitz, 2019/06/03
- Re: [Qemu-devel] [PATCH v2 1/2] qapi/block-core: Overlays are not snapshots,
Markus Armbruster <=
- Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/2] blockdev: Overlays are not snapshots, John Snow, 2019/06/03
- Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/2] blockdev: Overlays are not snapshots, Alberto Garcia, 2019/06/04
- Re: [Qemu-devel] [PATCH v2 0/2] blockdev: Overlays are not snapshots, Max Reitz, 2019/06/13