diff options
| author | Ben Pfaff <blp@nicira.com> | 2015-10-15 09:46:21 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2015-11-03 19:45:29 -0800 |
| commit | 68dfc25b62e5edc2939bcae791a35fddfecb5d20 (patch) | |
| tree | a64413e9309dabf569e1141b5c1ddef9a6c244af /tests/ofp-print.at | |
| parent | 337c452854451ed1b32d414a44c62b78b4a057c9 (diff) | |
| download | openvswitch-68dfc25b62e5edc2939bcae791a35fddfecb5d20.tar.gz | |
ofp-parse: Fix parsing, formatting of multiple fields in NTR extension.
Until now, the only way to specify multiple fields in the "fields"
parameter for the Netronome groups extension, was to specify "fields"
more than once, e.g. fields=eth_dst,fields=ip_dst
However, this wasn't documented and the code in ofp-print didn't use it,
generating output that couldn't be parsed.
This commit fixes the situation by introducing a more straightforward
syntax, e.g. fields(eth_dst,ip_dst), documents it, and adjusts ofp-print
code to use it when there is more than one field (it retains the previous
format for backward compatibility when there is exactly one field)
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Simon Horman <simon.horman@netronome.com>
Diffstat (limited to 'tests/ofp-print.at')
| -rw-r--r-- | tests/ofp-print.at | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/ofp-print.at b/tests/ofp-print.at index 25cf950c6..fce7671ff 100644 --- a/tests/ofp-print.at +++ b/tests/ofp-print.at @@ -2043,7 +2043,7 @@ ff ff 00 3b 00 00 15 40 00 00 00 01 00 00 00 00 \ 14 01 ff 00 00 00 00 00 \ "], [0], [dnl OFPST_GROUP_DESC reply (OF1.5) (xid=0x2): - group_id=8192,type=select,selection_method=hash,fields=ip_dst=255.255.255.0,nw_proto,tcp_src,bucket=bucket_id:0,weight:100,watch_port:1,actions=output:1,bucket=bucket_id:1,weight:200,watch_port:2,actions=output:2,bucket=bucket_id:2,weight:200,watch_port:3,actions=output:3 + group_id=8192,type=select,selection_method=hash,fields(ip_dst=255.255.255.0,nw_proto,tcp_src),bucket=bucket_id:0,weight:100,watch_port:1,actions=output:1,bucket=bucket_id:1,weight:200,watch_port:2,actions=output:2,bucket=bucket_id:2,weight:200,watch_port:3,actions=output:3 ]) AT_CLEANUP |