Change in ovirt-engine[master]: Due to commit a276f142 foreman id is not provided in vds sta...
masayag at redhat.com
masayag at redhat.com
Tue Jan 20 07:51:10 UTC 2015
Moti Asayag has submitted this change and it was merged.
Change subject: Due to commit a276f142 foreman id is not provided in vds static table
......................................................................
Due to commit a276f142 foreman id is not provided in vds static table
By Changing the order of adding vdsStatic db we miss the update of
setHostProviderId. This patch removes the duplication and use directly
vdsStatic to update provider id.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1113003
Change-Id: I163311d3c3e7edc89a0ca77a94eed688528ad192
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVdsActionParameters.java
2 files changed, 2 insertions(+), 5 deletions(-)
Approvals:
Yaniv Bronhaim: Verified
Moti Asayag: Looks good to me, approved
Yair Zaslavsky: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/37108
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I163311d3c3e7edc89a0ca77a94eed688528ad192
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list