)]}'
{"quantum/tests/unit/test_l3_plugin.py":[{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"beb34ffb98860a5cd0ae29842146f77a6e134057","unresolved":false,"context_lines":[{"line_number":452,"context_line":"                            \u0027add\u0027,"},{"line_number":453,"context_line":"                            r[\u0027router\u0027][\u0027id\u0027],"},{"line_number":454,"context_line":"                            s1[\u0027subnet\u0027][\u0027id\u0027],"},{"line_number":455,"context_line":"                            None)"},{"line_number":456,"context_line":"                        self._router_interface_action("},{"line_number":457,"context_line":"                            \u0027remove\u0027,"},{"line_number":458,"context_line":"                            r[\u0027router\u0027][\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2F6CQ%3D","line":455,"updated":"2012-10-29 16:14:17.000000000","message":"why do we check for port_id after the first interface add, but not here?  Seems inconsistent.   I think we should check the return value of the second add interface call to make sure it is not a 404, or am I missing some other way that this check is happening?","commit_id":"40232ca527b8f3ee81088c93160dd585d7bb6e6c"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c96219a9c80fcf6d683576116acf71c2f8c8510d","unresolved":false,"context_lines":[{"line_number":452,"context_line":"                            \u0027add\u0027,"},{"line_number":453,"context_line":"                            r[\u0027router\u0027][\u0027id\u0027],"},{"line_number":454,"context_line":"                            s1[\u0027subnet\u0027][\u0027id\u0027],"},{"line_number":455,"context_line":"                            None)"},{"line_number":456,"context_line":"                        self._router_interface_action("},{"line_number":457,"context_line":"                            \u0027remove\u0027,"},{"line_number":458,"context_line":"                            r[\u0027router\u0027][\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2F5vU%3D","line":455,"in_reply_to":"AAAALn%2F%2F6CQ%3D","updated":"2012-10-29 20:44:44.000000000","message":"Thanks for noticing it. I\u0027ll add port_id check. I dropped it by mistake during struggling with mock.","commit_id":"40232ca527b8f3ee81088c93160dd585d7bb6e6c"}]}