)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"68329af05aa4904b95f41c1b8b1dbb25aa91b5f5","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The correct method name should be `remove_endpoint_from_project`"},{"line_number":10,"context_line":"according to the definition in the controller layer or the name"},{"line_number":11,"context_line":"of abstract method."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I891a59969bcbbe5cc91a0dca3b8f1f253b93bcbc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba8a016a_c9a3929d","line":12,"updated":"2015-11-16 22:45:07.000000000","message":"This should have a bug since it\u0027s causing cache invalidation to fail.","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"}],"keystone/contrib/endpoint_filter/core.py":[{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"d692fd4ea7d8180a62e754835a564350868bb5f6","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_to_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_3b47944c","line":72,"updated":"2015-11-11 07:28:56.000000000","message":"need to change this too....the only reason this worked before is that the router call effectively bypassed this method and went straight to the driver...","commit_id":"69ec4e1a7187cb06cc8b25f59f5f6eb559eadd85"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"82fd70e242b61e9c0f9aa1028d5c39e5a48da4c9","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_to_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_97df6868","line":72,"in_reply_to":"da85f559_3b47944c","updated":"2015-11-11 12:58:30.000000000","message":"++","commit_id":"69ec4e1a7187cb06cc8b25f59f5f6eb559eadd85"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"cbe9556fb9c2a24dd4484a54a83a3397e966c6d6","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_to_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_8e2d05aa","line":72,"in_reply_to":"da85f559_3b47944c","updated":"2015-11-11 16:25:52.000000000","message":"Good catch!","commit_id":"69ec4e1a7187cb06cc8b25f59f5f6eb559eadd85"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"2d1645c4aac5da1677756c716021f1240f909709","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_from_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_d16ad2b2","line":72,"updated":"2015-11-11 16:27:28.000000000","message":"question, how did this not fail tests?","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"b902ee62403356a8e3c133bc97831ddd160b5d57","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_from_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_add32603","line":72,"in_reply_to":"da85f559_944433b7","updated":"2015-11-11 19:13:51.000000000","message":"ok, so there\u0027s a test missing. add the test.","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"cdcdb898cfbf5852ff2e9cb2379e9a1161cef990","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_from_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_e39cdf0f","line":72,"in_reply_to":"da85f559_add32603","updated":"2015-11-11 20:04:10.000000000","message":"Can we add the testcase in the following up review? this patch just correct the name of the method name, but we can address the testcase issue if testcase is missed.","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"be8972fddb55c29252965714290d1c4803005dd1","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_from_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_944433b7","line":72,"in_reply_to":"da85f559_b19aae6c","updated":"2015-11-11 18:13:16.000000000","message":"yep, i had to scratch my head to work out why as well!","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"de7241b7d042f17be57dbe26501bc0d5a9a25784","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_from_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_b19aae6c","line":72,"in_reply_to":"da85f559_d16ad2b2","updated":"2015-11-11 16:33:06.000000000","message":"I think henry should hold the answer in the earlier review :),  bypassed and go straightly to the call the method in the driver layer.","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"9bab63c69f5ce09607f5d03799fa1b4a9db36721","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def remove_endpoint_from_project(self, endpoint_id, project_id):"},{"line_number":72,"context_line":"        self.driver.remove_endpoint_from_project(endpoint_id, project_id)"},{"line_number":73,"context_line":"        catalog.COMPUTED_CATALOG_REGION.invalidate()"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def add_endpoint_group_to_project(self, endpoint_group_id, project_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_1e013a05","line":72,"in_reply_to":"da85f559_e39cdf0f","updated":"2015-11-11 20:07:47.000000000","message":"The testcase can be added in a follow-on review.","commit_id":"830ca0cb868f290e7990bf26bb62254bf36271b7"}]}