summaryrefslogtreecommitdiffhomepage
AgeCommit message (Collapse)Author
2012-05-21Add NXT_ROLE_REQUEST/REPLY constantsFUJITA Tomonori
NXT_ROLE_REQUEST/REPLY use the same on-wire format (NX_ROLE_PACK_STR). Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Isaku Yamahata <yamahata@valinux.co.jp>
2012-05-20ryu/controller/dpset.py: code simplificationIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-20lib/dpid: dpid formatter/parserIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-20Kill parser_stats_body_array method in OFPFlowStats classFUJITA Tomonori
OFPStatsReply needs to handle variable-length bodys so having two parsers (parser_stats_body_array and parser_stats_body) doesn't make sense. Just kill parser_stats_body_array(). cls_stats_body_cls_size is also pointless since OFPStatsReply needs to handle variable-length bodys. OFPStatsReply class needs to know if a stats class is array or not. So register_stats_type takes 'body_single_struct' instead of body_cls_size. We need to change this scheme if we need to handle VendorStats in the same way (both array and single struct). But currently we don't even have any VendorStats implementation so let's invent something more complicated when it becomes necessary. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Isaku Yamahata <yamahata@valinux.co.jp>
2012-05-20fix OFPQueueGetConfigReply parserFUJITA Tomonori
msg.port should not be a tuple. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-20fix OFPVendor parserFUJITA Tomonori
msg.vendor should not be tuple. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-20fix typo in ofproto_v1_0_parser commentFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-19fix StatsReply parserFUJITA Tomonori
OFPStatsReply parser_stats_body_array needs body classes to have length field. We could set length as class-wide constants for classes having a fixed length however we have two classes that need a variable length so let's set the length in the same way for all the stats classes. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-18Fix OFPQueuePropMinRate classFUJITA Tomonori
- __str__ method: needs to pass a tuple to msg_str_attr - parser method: rate shouldn't be a tuple. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-18fix OFPPacketQueue class parserFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-15ryu/app/wsapi: make json_parse_message_body workIsaku Yamahata
json_parse_message_body() didn't work. This patch fixes it. > File "ryu/app/wsapi.py", line 129, in json_parse_message_body > content = request.content.read() > AttributeError: WSRequest instance has no attribute 'content' > Traceback (most recent call last): Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-14Pretty NXAcationBundleBase parser upFUJITA Tomonori
Use '()' instead of '\' Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-14fix NXActionRegLoad parserFUJITA Tomonori
fix a syntax bug and typo. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-14fix NXActionRegMove parserFUJITA Tomonori
fix a syntax bug and typo. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-14fix NXActionMultipath parserFUJITA Tomonori
Fix a syntax bug. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-14add Nicira Extension NXAST_OUTPUT_REG supportFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Simon Horman <horms@verge.net.au>
2012-05-14add Nicira Extension NXAST_BUNDLE and NXAST_BUNDLE_LOAD supportFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-14add Nicira Extension NXAST_REG_LOAD supportFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Simon Horman <horms@verge.net.au>
2012-05-14add Nicira Extension NXAST_REG_MOVE supportFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Simon Horman <horms@verge.net.au>
2012-05-10Fix OFPStatsReply parserFUJITA Tomonori
ryu crashes when it gets OFPT_STATS_REPLY including OFP actions. ofp_flow_stats can include ofp_action_header(s) so parser_stats_body_array method can't use the entry_size argument (the minimum size, 88 bytes). It needs to use the size that body_cls's parser returns. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-08Add NXAction parser supportFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-08Add OFPActionStripVlan classFUJITA Tomonori
Avoid hitting the assert in OFPAction class' parser method. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07lib/mac: improve helper functionsIsaku Yamahata
- docstring - add sanity check as haddr_to_bin() is used to parse user-giving string Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07ofproto/ofproto_v1_0_parser: PacketIn parserIsaku Yamahata
frame data can contain padding of ofp packet. Discard padding. Otherwise packet parser can get confused which can rely on frame length. This is a preparation for LLDP TLV parser. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07ofproto_v1_0_parser: fix OFPPacketOut serializerIsaku Yamahata
0xffffffff == -1 as uint32_t. However python integer representation isn't sized integer. So assert buffer_id == -1 doesn't hold. use 0xffffffff explicitly instead of -1. Traceback (most recent call last): File "/usr/lib/pymodules/python2.7/gevent/greenlet.py", line 390, in run result = self._run(*self.args, **self.kwargs) File "/ryu/bin/ryu/app/discovery.py", line 421, in lldp_loop self.send_lldp_packet(dp, port_no) File "/ryu/bin/ryu/app/discovery.py", line 390, in send_lldp_packet dp.send_packet_out(actions=actions, data=port_data.data) File "/ryu/bin/ryu/controller/controller.py", line 211, in send_packet_out self.send_msg(packet_out) File "/ryu/bin/ryu/controller/controller.py", line 181, in send_msg self.serialize_msg(msg) File "/ryu/bin/ryu/controller/controller.py", line 178, in serialize_msg msg.serialize() File "/ryu/bin/ryu/ofproto/ofproto_parser.py", line 119, in serialize self._serialize_body() File "/ryu/bin/ryu/ofproto/ofproto_v1_0_parser.py", line 1210, in _serialize_body assert self.buffer_id == -1 AssertionError Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07remote COPYRIGHT file from MANIFEST.inIsaku Yamahata
711301abb3ab338e68411823e10ec40d0663d757 removed COPYRIGHT file So remove it from MANIFEST.in. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07README: added the pointer to Gmane ML archiveIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07add Nicira Extension NXAST_MULTIPATH actionFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07Added test framework, ported from QuantumFUJITA Tomonori
* Added test result format for Jenkins * Added some test code for ryu/ofproto/ Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07doc/source/step_by_step.rst: add item to setup quantum dbIsaku Yamahata
The step to create quantum db was missing. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-07Fix OFP_ACTION_DL_ADDR_PACK_STRFUJITA Tomonori
Use '6s' for a mac address. It's consistent with nx code and works with mac.haddr_to_bin helper function. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Simon Horman <horms@verge.net.au>
2012-05-07Fix OFPActionHeader class 'serialize' methodFUJITA Tomonori
OFPActionHeader class should have 'serialize' method. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> Reviewed-by: Simon Horman <horms@verge.net.au>
2012-05-01controller/controller: make send_flow_mod() allow default value for priorityIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01app: use symbol instead of 32678 for default priorityIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01ofproto/ofproto_v1_0_parser: flowmod default parameterIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01ofproto/ofproto_v1_0_parser: make FlowMod allow meaningful default parameterIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Reviewed-by: Simon Horman <simon@horms.net> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01controller, ofproto_v1_0: avoid format for argumentIsaku Yamahata
format is defined in builtin, so it should be avoid for argument. use flow_format instead. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Reviewed-by: Simon Horman <simon@horms.net> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01ofproto/ofproto_v1_0_parser: serializer for NXActionResbumitIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01ofproto/ofproto_v1_0_parser: serializer for nxt_flow_mod_table_idIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01ofproto/ofproto_v1_0_parser: improve NXTRequest initializerIsaku Yamahata
initialize subtype by base class, NXTRequest, instead of subclass. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Reviewed-by: Simon Horman <simon@horms.net> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01ofproto/ofproto_v1_0: more constants for nxIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01app/wsapi: make WSPathComponent inherit objectIsaku Yamahata
Later we create subclass of WSPathComponent so that make super() work. app/wsapi: super().__init__() Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01app/wsapi/WSPathArbitraryString: wrong indentIsaku Yamahata
WSPathArbitraryString.__str__ method should be instance method. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01add missing super().__init__()Isaku Yamahata
- app/event_dumper - controller/dispatcher - controller/mac_to_network - controller/mac_to_port - app/simple_switch Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-05-01app/simple_isolation: unbreak nx patchIsaku Yamahata
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-04-11fix find_flagfile path handling bugYoshihiro Kaneko
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-04-06update docs on the license switchFUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-04-06Switch to Apache 2.0 licenseFUJITA Tomonori
To make the further integration into IaaS OSS (OpenStack and CloudStack) easier, we switch to Apache 2.0 license. ryu/app/wsapi.py is still under GPL3 or later. We replace it later. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-04-06Use the original file licenseFUJITA Tomonori
This file is based on code distributed under GPL3 or later so it should stick to the same license. Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
2012-04-04ryu 0.2FUJITA Tomonori
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>