Muli Salem has uploaded a new change for review.

Change subject: core: Prevent Reporting UNKNOWN Network (#849163)
......................................................................

core: Prevent Reporting UNKNOWN Network (#849163)

https://bugzilla.redhat.com/849163

This patch makes sure the Network name is only removed from
the list of bad Networks, if the Bond status is true,
meaning we do not need the Network name to be printed out
to the Audit log.

Change-Id: I0a8349b23c3b578d33118113ff618740f1d9ab8e
Signed-off-by: Muli Salem <[email protected]>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/24/7424/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
index 4c9bcd9..27080f1 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
@@ -653,7 +653,7 @@
             // we remove the network from the audit log if the bond
             // is active
             else if (networks.contains(networkName)
-                    && bondsWithStatus.containsKey(iface.getBondName())) {
+                    && bondsWithStatus.containsKey(iface.getBondName()) && 
bondsWithStatus.get(iface.getBondName())) {
                 networks.remove(networkName);
             }
             if (!bondsWithListOfNics.containsKey(iface.getBondName())) {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a8349b23c3b578d33118113ff618740f1d9ab8e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to