)]}'
{"neutron/db/firewall/firewall_db.py":[{"author":{"_account_id":1923,"name":"Mohammad Banikazemi","email":"mbanikazemi@gmail.com","username":"mb-s"},"change_message_id":"e03283d737f2fb477b8700ba6191c36eed835b13","unresolved":false,"context_lines":[{"line_number":353,"context_line":"        dst_port_min, dst_port_max \u003d self._get_min_max_ports_from_range("},{"line_number":354,"context_line":"            fwr[\u0027destination_port\u0027])"},{"line_number":355,"context_line":"        if fwr[\u0027protocol\u0027] is None:"},{"line_number":356,"context_line":"            fwr[\u0027protocol\u0027] \u003d \"any\""},{"line_number":357,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":358,"context_line":"            fwr_db \u003d FirewallRule(id\u003duuidutils.generate_uuid(),"},{"line_number":359,"context_line":"                                  tenant_id\u003dtenant_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_d7d6a1f0","line":356,"updated":"2014-07-03 21:20:23.000000000","message":"This doesn\u0027t seem to be the right place. If anything, when the data is displayed the conversion should happen. I think the assumption is that if the protocol field is not shown it means \u0027any\u0027.","commit_id":"de36b8e668bc7fba65d75e3e92a4335ce9c1beef"},{"author":{"_account_id":11614,"name":"Koteswara Rao Kelam","email":"koteswara.kelam@hp.com","username":"koteswar"},"change_message_id":"3844f3212c0aed9349bdc5a75668ba69aeaf81b0","unresolved":false,"context_lines":[{"line_number":353,"context_line":"        dst_port_min, dst_port_max \u003d self._get_min_max_ports_from_range("},{"line_number":354,"context_line":"            fwr[\u0027destination_port\u0027])"},{"line_number":355,"context_line":"        if fwr[\u0027protocol\u0027] is None:"},{"line_number":356,"context_line":"            fwr[\u0027protocol\u0027] \u003d \"any\""},{"line_number":357,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":358,"context_line":"            fwr_db \u003d FirewallRule(id\u003duuidutils.generate_uuid(),"},{"line_number":359,"context_line":"                                  tenant_id\u003dtenant_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_64e35ea0","line":356,"in_reply_to":"baada198_d7d6a1f0","updated":"2014-07-25 08:25:47.000000000","message":"agree. abandon this change and submitted new pathch for neutronclient code. https://review.openstack.org/109521","commit_id":"de36b8e668bc7fba65d75e3e92a4335ce9c1beef"}]}
