Change in ovirt-engine[ovirt-engine-3.5]: Override firewall configurations on deploy for provisioned h...
Oved Ourfali has submitted this change and it was merged.
Change subject: Override firewall configurations on deploy for provisioned hosts
......................................................................
Override firewall configurations on deploy for provisioned hosts
Currently we don't have parameter for override-firewall configurations
in hosts\[id]\install restapi. Therefore we cannot pass this parameter along
the way from foreman to engine. Until adding the parameter to the API
this patch hacks the code to always override the firewall configurations
for vdsm deploy for each foreman installation when updateVdsCommand is
initiated by the reinstall request. This is monitored by checking that
the host in InstallingOS status, which can be only if the host is
firstly being provisioned by foreman.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1177126
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
Change-Id: I2634c33d87e9ef8ef41deeac28287b3b7ae1344a
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
1 file changed, 8 insertions(+), 1 deletion(-)
Approvals:
Yaniv Bronhaim: Verified
Oved Ourfali: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/36632
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2634c33d87e9ef8ef41deeac28287b3b7ae1344a
Gerrit-PatchSet: 6
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