Oved Ourfali 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
Oved Ourfali: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/37113
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I163311d3c3e7edc89a0ca77a94eed688528ad192
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server