# HG changeset patch
# User Rob Hoes <[email protected]>
CA-39188: Change illegal MTU value to 1500 when creating Network

There is already a default of 1500 in the datamodel for the Network.MTU field. 
However, language bindings tend to fill in their own default of 0 (which is 
illegal), when the field is not specified by the user.

Signed-off-by: Rob Hoes <[email protected]>

diff -r a8c9fb523f7e ocaml/xapi/xapi_network.ml
--- a/ocaml/xapi/xapi_network.ml        Thu Mar 25 09:49:35 2010 +0000
+++ b/ocaml/xapi/xapi_network.ml        Thu Mar 25 10:23:14 2010 +0000
@@ -124,6 +124,7 @@
        Mutex.execute mutex (fun () ->
                let networks = Db.Network.get_all ~__context in
                let bridges = List.map (fun self -> Db.Network.get_bridge 
~__context ~self) networks in
+               let mTU = if mTU <= 0L then 1500L else mTU in
                let rec loop () = 
                        let name = stem ^ (string_of_int !counter) in
                        incr counter;
# HG changeset patch
# User Rob Hoes <[email protected]>
CA-39188: Change illegal MTU value to 1500 when creating Network

There is already a default of 1500 in the datamodel for the Network.MTU field. 
However, language bindings tend to fill in their own default of 0 (which is 
illegal), when the field is not specified by the user.

Signed-off-by: Rob Hoes <[email protected]>

diff -r a8c9fb523f7e ocaml/xapi/xapi_network.ml
--- a/ocaml/xapi/xapi_network.ml        Thu Mar 25 09:49:35 2010 +0000
+++ b/ocaml/xapi/xapi_network.ml        Thu Mar 25 10:23:14 2010 +0000
@@ -124,6 +124,7 @@
        Mutex.execute mutex (fun () ->
                let networks = Db.Network.get_all ~__context in
                let bridges = List.map (fun self -> Db.Network.get_bridge 
~__context ~self) networks in
+               let mTU = if mTU <= 0L then 1500L else mTU in
                let rec loop () = 
                        let name = stem ^ (string_of_int !counter) in
                        incr counter;
_______________________________________________
xen-api mailing list
[email protected]
http://lists.xensource.com/mailman/listinfo/xen-api

Reply via email to