)]}'
{"dragonflow/controller/l2_ml2_app.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"ae563b57ba3ea2049e6c5db96ac70845a905d73d","unresolved":false,"context_lines":[{"line_number":684,"context_line":"        actions \u003d [parser.OFPActionSetField(tunnel_id_nxm\u003dsegmentation_id),"},{"line_number":685,"context_line":"                   parser.OFPActionOutput(port\u003dofport)]"},{"line_number":686,"context_line":""},{"line_number":687,"context_line":"        tunnels \u003d set(str(ofport))"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"        # todo"},{"line_number":690,"context_line":"        # aggregate  remote tunnel"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_7ea36700","line":687,"updated":"2016-07-27 12:27:10.000000000","message":"ofport is an int. str(ofport) is a string. set(str(ofport)) will be the set of all characters in ofport. If ofport\u003e\u003d10, this will lead to unexpected results.","commit_id":"6c53a41ee69529301117d0c29c6ac0dae17752dd"},{"author":{"_account_id":20405,"name":"liuhaixia","email":"liuhaixia315@huawei.com","username":"liuhaixia"},"change_message_id":"ad0db855e8351ee330a49fc80e7e414935265c8a","unresolved":false,"context_lines":[{"line_number":684,"context_line":"        actions \u003d [parser.OFPActionSetField(tunnel_id_nxm\u003dsegmentation_id),"},{"line_number":685,"context_line":"                   parser.OFPActionOutput(port\u003dofport)]"},{"line_number":686,"context_line":""},{"line_number":687,"context_line":"        tunnels \u003d set(str(ofport))"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"        # todo"},{"line_number":690,"context_line":"        # aggregate  remote tunnel"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_04dc5211","line":687,"in_reply_to":"bacf61ea_c4419a3b","updated":"2016-07-28 02:31:04.000000000","message":"Sorry, I know what you mean.\nI have fixed it.\nThank you.","commit_id":"6c53a41ee69529301117d0c29c6ac0dae17752dd"},{"author":{"_account_id":20405,"name":"liuhaixia","email":"liuhaixia315@huawei.com","username":"liuhaixia"},"change_message_id":"863c15f71a2ef0c4fa9f497ad6b0b7c53470e65c","unresolved":false,"context_lines":[{"line_number":684,"context_line":"        actions \u003d [parser.OFPActionSetField(tunnel_id_nxm\u003dsegmentation_id),"},{"line_number":685,"context_line":"                   parser.OFPActionOutput(port\u003dofport)]"},{"line_number":686,"context_line":""},{"line_number":687,"context_line":"        tunnels \u003d set(str(ofport))"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"        # todo"},{"line_number":690,"context_line":"        # aggregate  remote tunnel"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_c4419a3b","line":687,"in_reply_to":"dada55a8_7ea36700","updated":"2016-07-28 02:11:48.000000000","message":"For example, ofport\u003d11, set(str(ofport)) contains \u002711\u0027 , not \u00271\u0027\u00271\u0027. When I add a new ofport(12), set will be \u002711\u0027 \u002712\u0027.\nI made a test, I think it works.","commit_id":"6c53a41ee69529301117d0c29c6ac0dae17752dd"}]}