[node-patches] Change in ovirt-node[master]: network: Fix NIC.is_configured()

mburns at redhat.com mburns at redhat.com
Tue Jul 9 13:26:32 UTC 2013


Michael Burns has submitted this change and it was merged.

Change subject: network: Fix NIC.is_configured()
......................................................................


network: Fix NIC.is_configured()

Previously a NIC was only assumed to be configured if a bootprotocol was
given. But this is wrong for the static IP cases, when no bootprotocol
but an IP address are set.
Thus the logic is updated and a NIC is assumed to be configured when a
bootprotocol or an IP address is set.

The symptoms were that the NIC appeared as unconfigured in the network
page, when a static IP was set.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=980465
Change-Id: Ic2d3e2d4ddb75958dd7cbb67bb29a305cc6d2411
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/network.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved


-- 
To view, visit http://gerrit.ovirt.org/16507
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic2d3e2d4ddb75958dd7cbb67bb29a305cc6d2411
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list