Re: [libvirt] [PATCH v2 18/25] conf: backup: Add fields for tracking stats of completed sub-jobs

2019-12-09 Thread Ján Tomko

On Tue, Dec 03, 2019 at 06:17:40PM +0100, Peter Krempa wrote:

We need a place to store stats of completed sub-jobs so that we can
later report accurate stats.

Signed-off-by: Peter Krempa 
---
src/conf/backup_conf.h | 7 +++
1 file changed, 7 insertions(+)



Reviewed-by: Ján Tomko 

Jano


signature.asc
Description: PGP signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v2 18/25] conf: backup: Add fields for tracking stats of completed sub-jobs

2019-12-04 Thread Eric Blake

On 12/3/19 11:17 AM, Peter Krempa wrote:

We need a place to store stats of completed sub-jobs so that we can
later report accurate stats.

Signed-off-by: Peter Krempa 
---
  src/conf/backup_conf.h | 7 +++
  1 file changed, 7 insertions(+)

diff --git a/src/conf/backup_conf.h b/src/conf/backup_conf.h
index c970e01920..5dfc42e297 100644
--- a/src/conf/backup_conf.h
+++ b/src/conf/backup_conf.h
@@ -70,6 +70,13 @@ struct _virDomainBackupDef {

  size_t ndisks; /* should not exceed dom->ndisks */
  virDomainBackupDiskDef *disks;
+
+/* internal data */
+/* statistic totals for completed diks */


disks

With typo fix,
Reviewed-by: Eric Blake 

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.   +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

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



Re: [libvirt] [PATCH v2 18/25] conf: backup: Add fields for tracking stats of completed sub-jobs

2019-12-04 Thread Daniel P . Berrangé
On Tue, Dec 03, 2019 at 06:17:40PM +0100, Peter Krempa wrote:
> We need a place to store stats of completed sub-jobs so that we can
> later report accurate stats.

Its kind of gross using the public struct for internal only data
but I guess we can fix that any time we want to later.

> 
> Signed-off-by: Peter Krempa 
> ---
>  src/conf/backup_conf.h | 7 +++
>  1 file changed, 7 insertions(+)

Reviewed-by: Daniel P. Berrangé 

Regards,
Daniel
-- 
|: https://berrange.com  -o-https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o-https://fstop138.berrange.com :|
|: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|

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

[libvirt] [PATCH v2 18/25] conf: backup: Add fields for tracking stats of completed sub-jobs

2019-12-03 Thread Peter Krempa
We need a place to store stats of completed sub-jobs so that we can
later report accurate stats.

Signed-off-by: Peter Krempa 
---
 src/conf/backup_conf.h | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/src/conf/backup_conf.h b/src/conf/backup_conf.h
index c970e01920..5dfc42e297 100644
--- a/src/conf/backup_conf.h
+++ b/src/conf/backup_conf.h
@@ -70,6 +70,13 @@ struct _virDomainBackupDef {

 size_t ndisks; /* should not exceed dom->ndisks */
 virDomainBackupDiskDef *disks;
+
+/* internal data */
+/* statistic totals for completed diks */
+unsigned long long push_transferred;
+unsigned long long push_total;
+unsigned long long pull_tmp_used;
+unsigned long long pull_tmp_total;
 };

 typedef enum {
-- 
2.23.0

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