Change in ovirt-engine[master]: engine: Prevent running SetupNetworks as part of ApproveHost...

[alkaplan at redhat.com]

Alona Kaplan has submitted this change and it was merged.

Change subject: engine: Prevent running SetupNetworks as part of ApproveHost flow
......................................................................


engine: Prevent running SetupNetworks as part of ApproveHost flow

configureNetworks() method  should not be run as part of
ChangeVDSClusterCommand when it is executed as part of
host approval flow.
Configuring networks on the host is done later as part of InstallHost.

Change-Id: I7e9df62115654b0c36827ebf320b74e41d9cabd4
Bug-Url: https://bugzilla.redhat.com/1271273
Signed-off-by: Yevgeny Zaspitsky <yzaspits at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alona Kaplan: Passed CI tests
  Moti Asayag: Looks good to me, approved
  Yevgeny Zaspitsky: Verified



-- 
To view, visit https://gerrit.ovirt.org/50206
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7e9df62115654b0c36827ebf320b74e41d9cabd4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>