Change in ovirt-engine[master]: core: changed logic when should be 'default_route' flag pass...

[Code Review]

>From Alona Kaplan <alkaplan at redhat.com>:

Alona Kaplan has submitted this change and it was merged.

Change subject: core: changed logic when should be 'default_route' flag passed to vdsm
......................................................................


core: changed logic when should be 'default_route' flag passed to vdsm

Before default_route role default_route
flag was set when network was management network, and
had ipConfiguration data with gateway set in ipv4 or ipv6 address.

During upgrade script sooner in this topic we set all management
networks to have default_route role, so we can just check this.

Change-Id: Iaedb9375c68781c0fae52080bd4205b3cab031ea
Bug-Url: https://bugzilla.redhat.com/1200963
Signed-off-by: Martin Mucha <mmucha at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksCommand.java
1 file changed, 9 insertions(+), 6 deletions(-)

Approvals:
  Martin Mucha: Verified
  Alona Kaplan: Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaedb9375c68781c0fae52080bd4205b3cab031ea
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>