vz: fixes after commits refactoring common snapshot code

[libvirt] [PATCH 0/3] vz: fixes after commits refactoring common snapshot code

Nikolay Shirokovskiy posted 3 patches 5 years ago

Download series mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1554712973-336801-1-git-send-email-nshirokovskiy@virtuozzo.com

src/vz/vz_driver.c | 36 +++++++-----------------------------
src/vz/vz_sdk.c    | 24 +++++++++++-------------
2 files changed, 18 insertions(+), 42 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/3] vz: fixes after commits refactoring common snapshot code

Posted by Nikolay Shirokovskiy 5 years ago

Nikolay Shirokovskiy (3):
  vz: fix for tracking current snapshot
  vz: fixes: snapshot: Switch type of virDomainSnapshotObj.def
  vz: fixes: snapshot: Factor out virDomainMomentDef class

 src/vz/vz_driver.c | 36 +++++++-----------------------------
 src/vz/vz_sdk.c    | 24 +++++++++++-------------
 2 files changed, 18 insertions(+), 42 deletions(-)

-- 
1.8.3.1

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

Re: [libvirt] [PATCH 0/3] vz: fixes after commits refactoring common snapshot code

Posted by Cole Robinson 5 years ago

On 4/8/19 4:42 AM, Nikolay Shirokovskiy wrote:
> Nikolay Shirokovskiy (3):
>   vz: fix for tracking current snapshot
>   vz: fixes: snapshot: Switch type of virDomainSnapshotObj.def
>   vz: fixes: snapshot: Factor out virDomainMomentDef class
> 
>  src/vz/vz_driver.c | 36 +++++++-----------------------------
>  src/vz/vz_sdk.c    | 24 +++++++++++-------------
>  2 files changed, 18 insertions(+), 42 deletions(-)
> 

Untested, but looks fine to me

Reviewed-by: Cole Robinson <crobinso@redhat.com>

- Cole

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

Re: [libvirt] [PATCH 0/3] vz: fixes after commits refactoring common snapshot code

Posted by Maxim Nestratov 5 years ago

08-Apr-19 11:42, Nikolay Shirokovskiy пишет:

> Nikolay Shirokovskiy (3):
>    vz: fix for tracking current snapshot
>    vz: fixes: snapshot: Switch type of virDomainSnapshotObj.def
>    vz: fixes: snapshot: Factor out virDomainMomentDef class
>
>   src/vz/vz_driver.c | 36 +++++++-----------------------------
>   src/vz/vz_sdk.c    | 24 +++++++++++-------------
>   2 files changed, 18 insertions(+), 42 deletions(-)


ACK. Thanks.

Maxim Nestratov


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