Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 5: * Update tracker::#1273837::OK * Set MODIFIED::bug 1273837#1273837IGNORE, not o

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vmstats: take in account missing bulk stats fields .. vmstats: take in account missing bulk stats fields One of the cornerstones of how libvirt's bulk

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/47760 To unsubscribe, visit https:/

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/47760 To unsubscribe, visit https://

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 4: * Update tracker::#1273837::OK * Check Bug-Url::OK * Check Public Bug::#1273837::OK, pu

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 4: Verified+1 changes in the comments -> copying score -- To view, visit https://gerrit.ovir

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 3: (2 comments) https://gerrit.ovirt.org/#/c/47760/3/vdsm/virt/vmstats.py File vdsm/virt/vmst

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 3: Code-Review-1 (2 comments) Fine, just typos in the comment. https://gerrit.ovirt.org/#/c/477

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 3: Verified+1 this change is very simple and covered by tests. Verified running the testsuite.

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 3: * Update tracker::#1273837::OK * Check Bug-Url::OK * Check Public Bug::#1273837::OK, pu

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 2: please add Bug-Url: https://bugzilla.redhat.com/1273837 -- To view, visit https://gerrit.ovirt.

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/47760 To unsubscribe, visit https://ger

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, mak

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 1: an example would be: vcpu.0.time=2208000 net.count=2 net.1.name=vnet0 net.1.rx.bytes=64

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 1: Code-Review-1 I think it might be a good idea to fix _FAKE_BULK_STATS to simulate this in a unit

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: take in account missing bulk stats fields .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, mak

Change in vdsm[master]: vmstats: take in account missing bulk stats fields

2015-10-27 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vmstats: take in account missing bulk stats fields .. vmstats: take in account missing bulk stats fields One of the cornerstones of how libvirt's bulk stats