)]}'
{"quantum/plugins/nicira/nicira_nvp_plugin/QuantumPlugin.py":[{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"59b94ceba0ec58a8921f8437151565c36b15f862","unresolved":false,"context_lines":[{"line_number":1397,"context_line":"        return super(NvpPluginV2, self).update_router(context, id, router)"},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"    def delete_router(self, context, id):"},{"line_number":1400,"context_line":"        print \"NVP DELETE ROUTER\""},{"line_number":1401,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":1402,"context_line":"            super(NvpPluginV2, self).delete_router(context, id)"},{"line_number":1403,"context_line":"            # If removal is successful in Quantum it should be so on"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1o4%3D","line":1400,"updated":"2013-02-18 01:51:06.000000000","message":"looks like debug, should be removed.","commit_id":"ff80ba752316bce4d3f45344c9f4f7437e5ab575"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"1d7b18e5cf0219e423752b01daa6ce95e9648367","unresolved":false,"context_lines":[{"line_number":1397,"context_line":"        return super(NvpPluginV2, self).update_router(context, id, router)"},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"    def delete_router(self, context, id):"},{"line_number":1400,"context_line":"        print \"NVP DELETE ROUTER\""},{"line_number":1401,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":1402,"context_line":"            super(NvpPluginV2, self).delete_router(context, id)"},{"line_number":1403,"context_line":"            # If removal is successful in Quantum it should be so on"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1n0%3D","line":1400,"in_reply_to":"AAAAM3%2F%2B1o4%3D","updated":"2013-02-18 01:59:52.000000000","message":"Done","commit_id":"ff80ba752316bce4d3f45344c9f4f7437e5ab575"},{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"59b94ceba0ec58a8921f8437151565c36b15f862","unresolved":false,"context_lines":[{"line_number":1413,"context_line":"                              \"on NVP Platform\") % id)"},{"line_number":1414,"context_line":"            except NvpApiClient.NvpApiException:"},{"line_number":1415,"context_line":"                raise nvp_exc.NvpPluginException("},{"line_number":1416,"context_line":"                    err_msg\u003d(_(\"Unable to update logical router\""},{"line_number":1417,"context_line":"                               \"on NVP Platform\")))"},{"line_number":1418,"context_line":""},{"line_number":1419,"context_line":"    def get_router(self, context, id, fields\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1o0%3D","line":1416,"updated":"2013-02-18 01:51:06.000000000","message":"why does this say update?\n\nso is the general model that the delete will fail if we can\u0027t reach NVP, but if we reach NVP and it says the router doesn\u0027t exist, we will go ahead and delete?","commit_id":"ff80ba752316bce4d3f45344c9f4f7437e5ab575"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"1d7b18e5cf0219e423752b01daa6ce95e9648367","unresolved":false,"context_lines":[{"line_number":1413,"context_line":"                              \"on NVP Platform\") % id)"},{"line_number":1414,"context_line":"            except NvpApiClient.NvpApiException:"},{"line_number":1415,"context_line":"                raise nvp_exc.NvpPluginException("},{"line_number":1416,"context_line":"                    err_msg\u003d(_(\"Unable to update logical router\""},{"line_number":1417,"context_line":"                               \"on NVP Platform\")))"},{"line_number":1418,"context_line":""},{"line_number":1419,"context_line":"    def get_router(self, context, id, fields\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1nw%3D","line":1416,"in_reply_to":"AAAAM3%2F%2B1o0%3D","updated":"2013-02-18 01:59:52.000000000","message":"I\u0027m not sure why it says Update I believe it should say Delete.  Will update in another patch set. \n\nYup that\u0027s the exact model. If someone deletes the router directly from NVP this would still allow us to remove it in quantum.","commit_id":"ff80ba752316bce4d3f45344c9f4f7437e5ab575"},{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"59b94ceba0ec58a8921f8437151565c36b15f862","unresolved":false,"context_lines":[{"line_number":1417,"context_line":"                               \"on NVP Platform\")))"},{"line_number":1418,"context_line":""},{"line_number":1419,"context_line":"    def get_router(self, context, id, fields\u003dNone):"},{"line_number":1420,"context_line":"        print \"NVP GEt ROUTER\""},{"line_number":1421,"context_line":"        router \u003d self._get_router(context, id)"},{"line_number":1422,"context_line":"        try:"},{"line_number":1423,"context_line":"            # FIXME(salvatore-orlando): We need to"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1os%3D","line":1420,"updated":"2013-02-18 01:51:06.000000000","message":"remove","commit_id":"ff80ba752316bce4d3f45344c9f4f7437e5ab575"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"1d7b18e5cf0219e423752b01daa6ce95e9648367","unresolved":false,"context_lines":[{"line_number":1417,"context_line":"                               \"on NVP Platform\")))"},{"line_number":1418,"context_line":""},{"line_number":1419,"context_line":"    def get_router(self, context, id, fields\u003dNone):"},{"line_number":1420,"context_line":"        print \"NVP GEt ROUTER\""},{"line_number":1421,"context_line":"        router \u003d self._get_router(context, id)"},{"line_number":1422,"context_line":"        try:"},{"line_number":1423,"context_line":"            # FIXME(salvatore-orlando): We need to"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1nU%3D","line":1420,"in_reply_to":"AAAAM3%2F%2B1os%3D","updated":"2013-02-18 01:59:52.000000000","message":"Done","commit_id":"ff80ba752316bce4d3f45344c9f4f7437e5ab575"}],"quantum/plugins/nicira/nicira_nvp_plugin/common/exceptions.py":[{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"59b94ceba0ec58a8921f8437151565c36b15f862","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"class NvpOutOfSyncException(NvpPluginException):"},{"line_number":42,"context_line":"    message \u003d _(\"Quantum state has diverged from the networking backend!\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"class NvpNatRuleMismatch(NvpPluginException):"},{"line_number":46,"context_line":"    message \u003d _(\"While retrieving NAT rules, %(actual_rules)s were found \""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2B1oM%3D","side":"PARENT","line":43,"updated":"2013-02-18 01:51:06.000000000","message":"hooray!","commit_id":"3f66bd980b2a371f0599196263c9f73b2cc50eb2"}]}