Change in ovirt-engine[master]: engine: Remove unused code and simplify host events

Code Review gerrit at ovirt.org
Sun May 7 15:58:55 UTC 2017


>From Moti Asayag <masayag at redhat.com>:

Moti Asayag has submitted this change and it was merged.

Change subject: engine: Remove unused code and simplify host events
......................................................................


engine: Remove unused code and simplify host events

There was an incorrect assumption that ${Operation} property is expected
to be passed to the host event log. However, by checking the events
which are expecting for it, the events are being created at place and
not via the VdsCommand.createAlert() method.

Change-Id: I874ebbf99f025b67ffd7a10d2602a41e157dd842
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/pm/StartVdsCommandTest.java
3 files changed, 14 insertions(+), 49 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Moti Asayag: Verified



-- 
To view, visit https://gerrit.ovirt.org/76448
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I874ebbf99f025b67ffd7a10d2602a41e157dd842
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list