Change in vdsm[master]: bondopts: Add defaults for the 'arp_all_targets' option

osvoboda at redhat.com osvoboda at redhat.com
Thu Oct 30 10:51:25 UTC 2014


Ondřej Svoboda has uploaded a new change for review.

Change subject: bondopts: Add defaults for the 'arp_all_targets' option
......................................................................

bondopts: Add defaults for the 'arp_all_targets' option

This option would be output even if it had a default value, which is harmless
but not desired. netinfoTests:testGetBondingOptions, whose purpose is to alert
us that we are missing some defaults, should now pass again.

Change-Id: I26874003ab5fe4f4fbe92805ea78b7823b742074
Signed-off-by: Ondřej Svoboda <osvoboda at redhat.com>
---
M vdsm/bonding-defaults.json
1 file changed, 28 insertions(+), 21 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/37/34637/1

diff --git a/vdsm/bonding-defaults.json b/vdsm/bonding-defaults.json
index 52f2d6c..4d5afdc 100644
--- a/vdsm/bonding-defaults.json
+++ b/vdsm/bonding-defaults.json
@@ -7,6 +7,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -26,9 +30,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [
@@ -67,6 +68,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -86,9 +91,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [
@@ -127,6 +129,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -146,9 +152,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [
@@ -187,6 +190,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -206,9 +213,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [
@@ -247,6 +251,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -266,9 +274,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [
@@ -307,6 +312,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -326,9 +335,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [
@@ -367,6 +373,10 @@
         "all_slaves_active": [
             "0"
         ],
+        "arp_all_targets": [
+            "any",
+            "0"
+        ],
         "arp_interval": [
             "0"
         ],
@@ -386,9 +396,6 @@
             "0"
         ],
         "miimon": [
-            "0"
-        ],
-        "min_links": [
             "0"
         ],
         "mode": [


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I26874003ab5fe4f4fbe92805ea78b7823b742074
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>


More information about the vdsm-patches mailing list