Dan Kenigsberg has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 11: Code-Review+2
Seems like Milan's textual comments have been addressed
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 12:
* #1357798::Update tracker: OK
* Set MODIFIED::bug 1357798#1357798IGNORE, not all related patches
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: periodic: fix stats age reporting
..
vm: periodic: fix stats age reporting
We need to report the Vm responsiveness, because it is one
important
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 11: Verified+1
verified again with 61420 (same way as before)
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 11:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 10:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 9:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Milan Zamazal has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 8:
> Commit message was updated.
It's better now, thanks!
--
To view, visit https://gerrit.ovirt.org/61310
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 8:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 7:
(4 comments)
https://gerrit.ovirt.org/#/c/61310/7//COMMIT_MSG
Commit Message:
PS7, Line 24: it
> its
Milan Zamazal has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 7: Code-Review+1
(4 comments)
It looks fine to me, I just suggest improving the commit message.
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 7:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 6: Verified+1
verified again as before
--
To view, visit https://gerrit.ovirt.org/61310
To unsubscribe,
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 6:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Jenkins CI has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 5: Continuous-Integration+1
Propagate review hook: Continuous Integration value inherited from patch 4
--
To
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 4:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 5:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/61310/3/lib/vdsm/virt/sampling.py
File
Dan Kenigsberg has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 3:
(2 comments)
https://gerrit.ovirt.org/#/c/61310/3/lib/vdsm/virt/sampling.py
File
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/61310/3/lib/vdsm/virt/sampling.py
File
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 3: Verified+1
added one test, copying score
--
To view, visit https://gerrit.ovirt.org/61310
To
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/61310/3/lib/vdsm/virt/sampling.py
File
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 3:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 2: Verified+1
PS1->2
- changed one comment
- dropped unused code
If jenkins is happy, the verification
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 2:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Francesco Romani has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 1: Verified+1
verified
1. using the tests
2. as described in BZ: running one VM, SIGSTOP'ing QEMU, waited
gerrit-hooks has posted comments on this change.
Change subject: vm: periodic: fix stats age reporting
..
Patch Set 1:
* #1357798::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1357798::OK, public bug
* Check
Francesco Romani has uploaded a new change for review.
Change subject: vm: periodic: fix stats age reporting
..
vm: periodic: fix stats age reporting
The Vm class reports the monitor responsiveness in two ways:
1. if a libvirt
28 matches
Mail list logo
