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(a)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(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server