On Fri, 19 Jul 2013 15:32:05 +0900
YAMAMOTO Takashi <[email protected]> wrote:

> this set includes of-wire json tests and
> fixes for problems found by the test.
> 
> changes from the previous post includes:
>     - bug fixes
>     - add missing tests for some messages
>         OFPTableFeaturesStatsReply
>         OFPPortDescStatsRequest
>         OFPPortDescStatsReply
> 
> YAMAMOTO Takashi (9):
>   of13: fix OFP_TABLE_FEATURES_PACK_STR
>   of13: fix OFPTableFeaturesStats.parser
>   of-wire json: adapt OF13 OFPPortDescStatsReply
>   of-wire json: adapt OF13 OFPTableFeaturesStatsReply
>   of-wire json: adapt OF13 OFPMeterBand
>   of-wire json test: expected results
>   of-wire json test: a program to generate test data for unit tests
>   of-wire json test: generated *.packet
>   of-wire json test: unit test
> 
>  ryu/ofproto/ofproto_v1_3.py                        |    2 +-
>  ryu/ofproto/ofproto_v1_3_parser.py                 |   53 +-
>  .../packet_data/of10/1-1-ofp_packet_out.packet     |  Bin 0 -> 122 bytes
>  ryu/tests/packet_data/of10/1-2-ofp_flow_mod.packet |  Bin 0 -> 80 bytes
>  .../packet_data/of10/1-4-ofp_packet_in.packet      |  Bin 0 -> 60 bytes
>  .../packet_data/of10/1-5-features_request.packet   |  Bin 0 -> 8 bytes
>  .../of10/1-6-ofp_switch_features.packet            |  Bin 0 -> 128 bytes
>  .../of12/3-0-ofp_desc_stats_reply.packet           |  Bin 0 -> 1072 bytes
>  .../packet_data/of12/3-1-ofp_packet_out.packet     |  Bin 0 -> 138 bytes
>  ryu/tests/packet_data/of12/3-10-ofp_hello.packet   |  Bin 0 -> 8 bytes
>  .../of12/3-11-ofp_flow_stats_request.packet        |  Bin 0 -> 56 bytes
>  .../of12/3-12-ofp_flow_stats_reply.packet          |  Bin 0 -> 320 bytes
>  .../packet_data/of12/3-13-ofp_echo_request.packet  |  Bin 0 -> 12 bytes
>  .../packet_data/of12/3-14-ofp_echo_reply.packet    |  Bin 0 -> 12 bytes
>  .../packet_data/of12/3-15-ofp_error_msg.packet     |  Bin 0 -> 20 bytes
>  .../packet_data/of12/3-16-ofp_experimenter.packet  |  Bin 0 -> 20 bytes
>  .../of12/3-17-ofp_barrier_request.packet           |  Bin 0 -> 8 bytes
>  .../packet_data/of12/3-18-ofp_barrier_reply.packet |  Bin 0 -> 8 bytes
>  .../packet_data/of12/3-19-ofp_role_request.packet  |  Bin 0 -> 24 bytes
>  ryu/tests/packet_data/of12/3-2-ofp_flow_mod.packet |  Bin 0 -> 88 bytes
>  .../packet_data/of12/3-20-ofp_role_reply.packet    |  Bin 0 -> 24 bytes
>  .../packet_data/of12/3-21-ofp_group_mod.packet     |  Bin 0 -> 48 bytes
>  .../packet_data/of12/3-22-ofp_port_mod.packet      |  Bin 0 -> 40 bytes
>  .../packet_data/of12/3-23-ofp_table_mod.packet     |  Bin 0 -> 16 bytes
>  .../of12/3-24-ofp_desc_stats_request.packet        |  Bin 0 -> 16 bytes
>  .../of12/3-25-ofp_aggregate_stats_request.packet   |  Bin 0 -> 56 bytes
>  .../of12/3-26-ofp_aggregate_stats_reply.packet     |  Bin 0 -> 40 bytes
>  .../of12/3-27-ofp_table_stats_request.packet       |  Bin 0 -> 16 bytes
>  .../of12/3-28-ofp_table_stats_reply.packet         |  Bin 0 -> 32656 bytes
>  .../of12/3-29-ofp_port_stats_request.packet        |  Bin 0 -> 24 bytes
>  ryu/tests/packet_data/of12/3-3-ofp_flow_mod.packet |  Bin 0 -> 80 bytes
>  .../of12/3-30-ofp_port_stats_reply.packet          |  Bin 0 -> 224 bytes
>  .../3-31-ofp_group_features_stats_request.packet   |  Bin 0 -> 16 bytes
>  .../3-32-ofp_group_features_stats_reply.packet     |  Bin 0 -> 56 bytes
>  .../of12/3-33-ofp_group_desc_stats_request.packet  |  Bin 0 -> 16 bytes
>  .../of12/3-34-ofp_group_desc_stats_reply.packet    |  Bin 0 -> 56 bytes
>  .../of12/3-35-ofp_queue_get_config_request.packet  |  Bin 0 -> 16 bytes
>  .../of12/3-36-ofp_queue_get_config_reply.packet    |  Bin 0 -> 112 bytes
>  .../of12/3-37-ofp_queue_stats_request.packet       |  Bin 0 -> 24 bytes
>  .../of12/3-38-ofp_queue_stats_reply.packet         |  Bin 0 -> 112 bytes
>  .../packet_data/of12/3-39-ofp_port_status.packet   |  Bin 0 -> 80 bytes
>  .../packet_data/of12/3-4-ofp_packet_in.packet      |  Bin 0 -> 140 bytes
>  .../packet_data/of12/3-40-ofp_flow_removed.packet  |  Bin 0 -> 64 bytes
>  .../of12/3-41-ofp_error_msg_experimenter.packet    |  Bin 0 -> 27 bytes
>  .../of12/3-5-ofp_features_request.packet           |  Bin 0 -> 8 bytes
>  .../packet_data/of12/3-6-ofp_features_reply.packet |  Bin 0 -> 160 bytes
>  .../packet_data/of12/3-7-ofp_set_config.packet     |  Bin 0 -> 12 bytes
>  .../of12/3-8-ofp_get_config_request.packet         |  Bin 0 -> 8 bytes
>  .../of12/3-9-ofp_get_config_reply.packet           |  Bin 0 -> 12 bytes
>  .../packet_data/of13/4-0-ofp_desc_reply.packet     |  Bin 0 -> 1072 bytes
>  .../packet_data/of13/4-1-ofp_packet_out.packet     |  Bin 0 -> 138 bytes
>  ryu/tests/packet_data/of13/4-10-ofp_hello.packet   |  Bin 0 -> 16 bytes
>  .../of13/4-11-ofp_flow_stats_request.packet        |  Bin 0 -> 56 bytes
>  .../of13/4-12-ofp_flow_stats_reply.packet          |  Bin 0 -> 320 bytes
>  .../packet_data/of13/4-13-ofp_echo_request.packet  |  Bin 0 -> 12 bytes
>  .../packet_data/of13/4-14-ofp_echo_reply.packet    |  Bin 0 -> 12 bytes
>  .../packet_data/of13/4-15-ofp_error_msg.packet     |  Bin 0 -> 20 bytes
>  .../packet_data/of13/4-16-ofp_experimenter.packet  |  Bin 0 -> 20 bytes
>  .../of13/4-17-ofp_barrier_request.packet           |  Bin 0 -> 8 bytes
>  .../packet_data/of13/4-18-ofp_barrier_reply.packet |  Bin 0 -> 8 bytes
>  .../packet_data/of13/4-19-ofp_role_request.packet  |  Bin 0 -> 24 bytes
>  ryu/tests/packet_data/of13/4-2-ofp_flow_mod.packet |  Bin 0 -> 88 bytes
>  .../packet_data/of13/4-20-ofp_role_reply.packet    |  Bin 0 -> 24 bytes
>  .../packet_data/of13/4-21-ofp_group_mod.packet     |  Bin 0 -> 48 bytes
>  .../packet_data/of13/4-22-ofp_port_mod.packet      |  Bin 0 -> 40 bytes
>  .../packet_data/of13/4-23-ofp_table_mod.packet     |  Bin 0 -> 16 bytes
>  .../packet_data/of13/4-24-ofp_desc_request.packet  |  Bin 0 -> 16 bytes
>  .../of13/4-25-ofp_aggregate_stats_request.packet   |  Bin 0 -> 56 bytes
>  .../of13/4-26-ofp_aggregate_stats_reply.packet     |  Bin 0 -> 40 bytes
>  .../of13/4-27-ofp_table_stats_request.packet       |  Bin 0 -> 16 bytes
>  .../of13/4-28-ofp_table_stats_reply.packet         |  Bin 0 -> 64 bytes
>  .../of13/4-29-ofp_port_stats_request.packet        |  Bin 0 -> 24 bytes
>  ryu/tests/packet_data/of13/4-3-ofp_flow_mod.packet |  Bin 0 -> 80 bytes
>  .../of13/4-30-ofp_port_stats_reply.packet          |  Bin 0 -> 240 bytes
>  .../of13/4-31-ofp_group_features_request.packet    |  Bin 0 -> 16 bytes
>  .../of13/4-32-ofp_group_features_reply.packet      |  Bin 0 -> 56 bytes
>  .../of13/4-33-ofp_group_desc_request.packet        |  Bin 0 -> 16 bytes
>  .../of13/4-34-ofp_group_desc_reply.packet          |  Bin 0 -> 56 bytes
>  .../of13/4-35-ofp_queue_get_config_request.packet  |  Bin 0 -> 16 bytes
>  .../of13/4-36-ofp_queue_get_config_reply.packet    |  Bin 0 -> 112 bytes
>  .../of13/4-37-ofp_queue_stats_request.packet       |  Bin 0 -> 24 bytes
>  .../of13/4-38-ofp_queue_stats_reply.packet         |  Bin 0 -> 136 bytes
>  .../packet_data/of13/4-39-ofp_port_status.packet   |  Bin 0 -> 80 bytes
>  .../packet_data/of13/4-4-ofp_packet_in.packet      |  Bin 0 -> 148 bytes
>  .../packet_data/of13/4-40-ofp_flow_removed.packet  |  Bin 0 -> 64 bytes
>  .../of13/4-42-ofp_get_async_request.packet         |  Bin 0 -> 8 bytes
>  .../of13/4-43-ofp_get_async_reply.packet           |  Bin 0 -> 32 bytes
>  .../packet_data/of13/4-44-ofp_set_async.packet     |  Bin 0 -> 32 bytes
>  .../packet_data/of13/4-46-ofp_flow_mod.packet      |  Bin 0 -> 96 bytes
>  .../of13/4-47-ofp_meter_config_request.packet      |  Bin 0 -> 24 bytes
>  .../of13/4-48-ofp_meter_config_reply.packet        |  Bin 0 -> 40 bytes
>  .../of13/4-49-ofp_meter_stats_request.packet       |  Bin 0 -> 24 bytes
>  .../of13/4-5-ofp_features_request.packet           |  Bin 0 -> 8 bytes
>  .../of13/4-50-ofp_meter_stats_reply.packet         |  Bin 0 -> 72 bytes
>  .../of13/4-51-ofp_meter_features_request.packet    |  Bin 0 -> 16 bytes
>  .../of13/4-52-ofp_meter_features_reply.packet      |  Bin 0 -> 32 bytes
>  .../of13/4-53-ofp_port_desc_request.packet         |  Bin 0 -> 16 bytes
>  .../of13/4-54-ofp_port_desc_reply.packet           |  Bin 0 -> 144 bytes
>  .../of13/4-56-ofp_table_features_reply.packet      |  Bin 0 -> 11072 bytes
>  .../packet_data/of13/4-6-ofp_features_reply.packet |  Bin 0 -> 32 bytes
>  .../packet_data/of13/4-7-ofp_set_config.packet     |  Bin 0 -> 12 bytes
>  .../of13/4-8-ofp_get_config_request.packet         |  Bin 0 -> 8 bytes
>  .../of13/4-9-ofp_get_config_reply.packet           |  Bin 0 -> 12 bytes
>  ryu/tests/packet_data_generator/Makefile           |    5 +
>  ryu/tests/packet_data_generator/rebar.config       |   32 +
>  ryu/tests/packet_data_generator/src/er.app.src     |   24 +
>  ryu/tests/packet_data_generator/src/x.erl          |   63 +
>  ryu/tests/packet_data_generator/src/x1.erl         |   90 +
>  ryu/tests/packet_data_generator/src/x3.erl         | 8539 
> ++++++++++++++++++++
>  ryu/tests/packet_data_generator/src/x4.erl         | 2310 ++++++

Applied, but the diff is incorrect? I can't find erl code. Having only
packet data (that is, no generator code in our repo) is totally fine by
me though.

------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to