Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 4: Verified+1
Verified by running latest engine and vdsm and by creating and suspending a
automat...@ovirt.org has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
Dan Kenigsberg has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/42579
To unsubscribe, visit
Dan Kenigsberg has submitted this change and it was merged.
Change subject: events: make sure to send only one event per status
..
events: make sure to send only one event per status
Initially we made an assumption that engine
Francesco Romani has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 4: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/42579
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 3: Code-Review+1
(1 comment)
looks ok, thanks for the changes. Minor comment inside.
Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/42579/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
automat...@ovirt.org has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found,
Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2:
(4 comments)
https://gerrit.ovirt.org/#/c/42579/2//COMMIT_MSG
Commit Message:
Line 6:
automat...@ovirt.org has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found,
Francesco Romani has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2:
(4 comments)
-1 for visibility/discussion
I'd prefer to notify(), and log, without the
Arik Hadas has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2: Verified+1
(1 comment)
got one event for Down (the bug on engine side where the event is
Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/42579/2//COMMIT_MSG
Commit Message:
Line 10:
Dan Kenigsberg has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/42579/2//COMMIT_MSG
Commit Message:
Line 10:
Piotr Kliczewski has uploaded a new change for review.
Change subject: events: make sure to send only one event per status
..
events: make sure to send only one event per status
Initially we made an assumption that engine can
automat...@ovirt.org has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found,
automat...@ovirt.org has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found,
Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/42579/1//COMMIT_MSG
Commit Message:
Line 7:
Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 1: Verified+1
Verified by running latest vdsm and the engine and checking that there was
Piotr Kliczewski has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 2: Verified+1
Verified by running latest engine and vdsm and stopping a vm to see that event
Dan Kenigsberg has posted comments on this change.
Change subject: events: make sure to send only one event per status
..
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/42579/1//COMMIT_MSG
Commit Message:
Line 7:
21 matches
Mail list logo