vswitch: Fix types of NetFlow columns in db schema.
authorBen Pfaff <blp@nicira.com>
Thu, 7 Jan 2010 23:26:36 +0000 (15:26 -0800)
committerBen Pfaff <blp@nicira.com>
Tue, 26 Jan 2010 17:46:42 +0000 (09:46 -0800)
This bug was masked by the translation done by ovsdb-idlc, which doesn't
fully validate its input.  In particular, it ignores fields that it doesn't
know about and drops them when it does translation, so nothing later can
find the problem.

An upcoming commit will ensure that it can't happen again, by dropping the
ovsdb-idlc translation step.

vswitchd/vswitch-idl.ovsidl

index 27804d4..128e8f1 100644 (file)
          "type": {"key": "string", "min": 1, "max": "unlimited"}},
        "engine_type": {
          "comment": "Engine type to use in NetFlow messages.  Defaults to datapath index if not specified.",
-         "type": "integer", "min":0, "max":1},
+         "type": {"key": "integer", "min": 0, "max": 1}},
        "engine_id": {
          "comment": "Engine ID to use in NetFlow messages.  Defaults to datapath index if not specified.",
-         "type": "integer", "min":0, "max":1},
+         "type": {"key": "integer", "min": 0, "max": 1}},
        "add_id_to_interface": {
          "comment": "Place least-significant 7 bits of engine ID into most significant bits of ingress and egress interface fields of NetFlow records?",
          "type": "boolean"},