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