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