Change in ovirt-engine[master]: engine: Update specific fields in SetVdsStatusVDSCommand

oourfali at redhat.com oourfali at redhat.com
Thu Jun 4 05:08:23 UTC 2015


Oved Ourfali has submitted this change and it was merged.

Change subject: engine: Update specific fields in SetVdsStatusVDSCommand
......................................................................


engine: Update specific fields in SetVdsStatusVDSCommand

The command should concern only about the host status and the
reason fields, and shouldn't update the entire host dynamic record
which lead to an unexpected result in certain flows, such described
in the bug.

Change-Id: I6322fc882eb85ceb143e6184a61428a7e4c0fb4d
Bug-Url: https://bugzilla.redhat.com/1213809
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDynamicDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDynamicDAODbFacadeImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDynamicDAOTest.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/SetVdsStatusVDSCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
M packaging/dbscripts/vds_sp.sql
7 files changed, 67 insertions(+), 10 deletions(-)

Approvals:
  Jenkins CI: Verified; Looks good to me, but someone else must approve
  Liran Zelkha: Looks good to me, but someone else must approve
  Moti Asayag: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6322fc882eb85ceb143e6184a61428a7e4c0fb4d
Gerrit-PatchSet: 3
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: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list