)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"c63725c706d752a2ce344feeb9184c9d14ce2b6d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"csum option for the vxlan/gre tunnels. The default is maintained as False."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I18dcd8946b585e70f8890a5c222ea37059c4a0c5"},{"line_number":13,"context_line":"Closes-bug: #1492111"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_5f3d5629","line":13,"updated":"2015-09-06 16:58:21.000000000","message":"Please add:\n\nImplements: bp ovs-tunnel-csum-option","commit_id":"f54f7825150815aefaabceeedbf5de1b1b530cce"},{"author":{"_account_id":12223,"name":"Mamta Prabhu","email":"mamtaprabhu@in.ibm.com","username":"mamtap"},"change_message_id":"1f19f5271375fca051ac329688d8816d32d32cb8","unresolved":false,"context_lines":[{"line_number":10,"context_line":"csum option for the vxlan/gre tunnels. The default is maintained as False."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I18dcd8946b585e70f8890a5c222ea37059c4a0c5"},{"line_number":13,"context_line":"Closes-bug: #1492111"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_7f04d913","line":13,"in_reply_to":"da20952f_5f3d5629","updated":"2015-09-06 17:53:48.000000000","message":"Done","commit_id":"f54f7825150815aefaabceeedbf5de1b1b530cce"}],"etc/neutron/plugins/ml2/openvswitch_agent.ini":[{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"53846573e2cbef42195781a79e5017ece8db1986","unresolved":false,"context_lines":[{"line_number":166,"context_line":"# (BoolOpt) Set or un-set the checksum on outgoing IP packet"},{"line_number":167,"context_line":"# carrying GRE/VXLAN tunnel. The default value is False."},{"line_number":168,"context_line":"#"},{"line_number":169,"context_line":"# tunnel_csum \u003d True"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"[securitygroup]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"da20952f_cb7e91e3","line":169,"updated":"2015-09-06 02:20:17.000000000","message":"we use:\n\n# tunnel_csum \u003d False \nto indicate the default value, to be consistent with other options.","commit_id":"57dab42ca3de20e366fe03319771ed7c8bd0bc8e"},{"author":{"_account_id":12223,"name":"Mamta Prabhu","email":"mamtaprabhu@in.ibm.com","username":"mamtap"},"change_message_id":"fa4cd9523dbd9deade80623e91a4d148debf9625","unresolved":false,"context_lines":[{"line_number":166,"context_line":"# (BoolOpt) Set or un-set the checksum on outgoing IP packet"},{"line_number":167,"context_line":"# carrying GRE/VXLAN tunnel. The default value is False."},{"line_number":168,"context_line":"#"},{"line_number":169,"context_line":"# tunnel_csum \u003d True"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"[securitygroup]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"da20952f_7d9dca08","line":169,"in_reply_to":"da20952f_cb7e91e3","updated":"2015-09-06 07:40:33.000000000","message":"Thank you. I have updated the default in the ini to False to keep it consistent with other options","commit_id":"57dab42ca3de20e366fe03319771ed7c8bd0bc8e"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"c63725c706d752a2ce344feeb9184c9d14ce2b6d","unresolved":false,"context_lines":[{"line_number":166,"context_line":"# (BoolOpt) Set or un-set the checksum on outgoing IP packet"},{"line_number":167,"context_line":"# carrying GRE/VXLAN tunnel. The default value is False."},{"line_number":168,"context_line":"#"},{"line_number":169,"context_line":"# tunnel_csum \u003d False"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"[securitygroup]"}],"source_content_type":"text/x-properties","patch_set":2,"id":"da20952f_bf14b29f","line":169,"updated":"2015-09-06 16:58:21.000000000","message":"I assume the default value is the same as is currently handled without this option?","commit_id":"f54f7825150815aefaabceeedbf5de1b1b530cce"},{"author":{"_account_id":12223,"name":"Mamta Prabhu","email":"mamtaprabhu@in.ibm.com","username":"mamtap"},"change_message_id":"44c6cf35c74b8f484648a1474ed67c8c11bcad41","unresolved":false,"context_lines":[{"line_number":166,"context_line":"# (BoolOpt) Set or un-set the checksum on outgoing IP packet"},{"line_number":167,"context_line":"# carrying GRE/VXLAN tunnel. The default value is False."},{"line_number":168,"context_line":"#"},{"line_number":169,"context_line":"# tunnel_csum \u003d False"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"[securitygroup]"}],"source_content_type":"text/x-properties","patch_set":2,"id":"da20952f_dff3a671","line":169,"in_reply_to":"da20952f_bf14b29f","updated":"2015-09-06 17:25:50.000000000","message":"Hi Kyle,\n\nBefore these changes, one would not see the option \u0027csum\u0027 in the ovs tunnel.\nBut now, even if the value is not changed, one will see the csum field in the tunnel.\ne.g \n options: {csum\u003d\"false\", df_default\u003d\"true\", in_key\u003dflow, local_ip\u003d..... }\n\ngongysh provided me with the following comment.\nthinking about if we should use csum option if it is false.\n\nSo, I am making the changes such that, only when the option is true, add this additional \"option:csum\" else ignore.\n\nI am hoping this is the information that you were asking for.","commit_id":"f54f7825150815aefaabceeedbf5de1b1b530cce"}],"neutron/agent/common/ovs_lib.py":[{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"390fc82824acb43de3108168ef586dbf3dc2b802","unresolved":false,"context_lines":[{"line_number":315,"context_line":"        options[\u0027local_ip\u0027] \u003d local_ip"},{"line_number":316,"context_line":"        options[\u0027in_key\u0027] \u003d \u0027flow\u0027"},{"line_number":317,"context_line":"        options[\u0027out_key\u0027] \u003d \u0027flow\u0027"},{"line_number":318,"context_line":"        options[\u0027csum\u0027] \u003d str(tunnel_csum).lower()"},{"line_number":319,"context_line":"        attrs.append((\u0027options\u0027, options))"},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"        return self.add_port(port_name, *attrs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_10e1152a","line":318,"updated":"2015-09-06 08:11:09.000000000","message":"thinking about if we should use csum option if it is false.","commit_id":"f54f7825150815aefaabceeedbf5de1b1b530cce"}]}