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@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@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server