From 4d2638791024e5b7eb71047c4416793c1d74023c Mon Sep 17 00:00:00 2001 From: IWASE Yusuke Date: Mon, 27 Nov 2017 15:22:01 +0900 Subject: ofproto_v1_5_parser: Change default of command_bucket_id Currently, Ryu will send the OFPGroupMod message with "command_bucket_id=0" by the default if using the OpenFlow version 1.5. OTOH, Open vSwitch expects "command_bucket_id=OFPG_BUCKET_ALL" if "command" is other than OFPGC_INSERT_BUCKET or OFPGC_REMOVE_BUCKET, otherwise OVS will rejects the OFPGroupMod message. This patch changes the default value of "command_bucket_id" to OFPG_BUCKET_ALL as OVS expects. Signed-off-by: IWASE Yusuke Signed-off-by: FUJITA Tomonori --- ryu/ofproto/ofproto_v1_5_parser.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ryu/ofproto/ofproto_v1_5_parser.py b/ryu/ofproto/ofproto_v1_5_parser.py index b61e6fef..0fbb5c03 100644 --- a/ryu/ofproto/ofproto_v1_5_parser.py +++ b/ryu/ofproto/ofproto_v1_5_parser.py @@ -6131,7 +6131,8 @@ class OFPGroupMod(MsgBase): datapath.send_msg(req) """ def __init__(self, datapath, command=ofproto.OFPGC_ADD, - type_=ofproto.OFPGT_ALL, group_id=0, command_bucket_id=0, + type_=ofproto.OFPGT_ALL, group_id=0, + command_bucket_id=ofproto.OFPG_BUCKET_ALL, buckets=None, properties=None, bucket_array_len=None): buckets = buckets if buckets else [] properties = properties if properties else [] -- cgit v1.2.3