)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"7f5cdf93a076116b57a1fb9219471d2ff5bef00c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Drop OfficialClientManager from tenant isolation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Scenario tests uses the tempest clients, so drop support for"},{"line_number":10,"context_line":"OfficialClientManager from the tenant isolation classes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I84638210043d376b4f4025a69751536651991293"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baa201ad_32dbecee","line":9,"updated":"2014-09-30 06:35:12.000000000","message":"very nit, s/uses/use ?","commit_id":"44d07ca0b107c9e360d80c59d1702d5e32821128"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"8a32059214a738d6a2611b3d26de6a7b5fa2d6c2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Drop OfficialClientManager from tenant isolation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Scenario tests uses the tempest clients, so drop support for"},{"line_number":10,"context_line":"OfficialClientManager from the tenant isolation classes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I84638210043d376b4f4025a69751536651991293"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baa201ad_2858ca31","line":9,"in_reply_to":"baa201ad_32dbecee","updated":"2014-09-30 20:57:46.000000000","message":"Done","commit_id":"44d07ca0b107c9e360d80c59d1702d5e32821128"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"7f5cdf93a076116b57a1fb9219471d2ff5bef00c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"OfficialClientManager from the tenant isolation classes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I84638210043d376b4f4025a69751536651991293"},{"line_number":13,"context_line":"Partially-implements: bp/tempest-client-scenarios"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baa201ad_12d7c8c2","line":13,"updated":"2014-09-30 06:35:12.000000000","message":"very nit. This should be bp:tempest-client-scenarios for the link of the BP.","commit_id":"44d07ca0b107c9e360d80c59d1702d5e32821128"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"8a32059214a738d6a2611b3d26de6a7b5fa2d6c2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"OfficialClientManager from the tenant isolation classes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I84638210043d376b4f4025a69751536651991293"},{"line_number":13,"context_line":"Partially-implements: bp/tempest-client-scenarios"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baa201ad_485d0e21","line":13,"in_reply_to":"baa201ad_12d7c8c2","updated":"2014-09-30 20:57:46.000000000","message":"Done","commit_id":"44d07ca0b107c9e360d80c59d1702d5e32821128"}],"tempest/common/isolated_creds.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"7f5cdf93a076116b57a1fb9219471d2ff5bef00c","unresolved":false,"context_lines":[{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    def _cleanup_default_secgroup(self, tenant):"},{"line_number":305,"context_line":"        net_client \u003d self.network_admin_client"},{"line_number":306,"context_line":"        resp, resp_body \u003d net_client.list_security_groups(tenant_id\u003dtenant,"},{"line_number":307,"context_line":"                                                          name\u003d\"default\")"},{"line_number":308,"context_line":"        secgroups_to_delete \u003d resp_body[\u0027security_groups\u0027]"},{"line_number":309,"context_line":"        for secgroup in secgroups_to_delete:"}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_f29fa401","line":306,"updated":"2014-09-30 06:35:12.000000000","message":"Why don\u0027t you change resp to _ here?","commit_id":"44d07ca0b107c9e360d80c59d1702d5e32821128"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"8a32059214a738d6a2611b3d26de6a7b5fa2d6c2","unresolved":false,"context_lines":[{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    def _cleanup_default_secgroup(self, tenant):"},{"line_number":305,"context_line":"        net_client \u003d self.network_admin_client"},{"line_number":306,"context_line":"        resp, resp_body \u003d net_client.list_security_groups(tenant_id\u003dtenant,"},{"line_number":307,"context_line":"                                                          name\u003d\"default\")"},{"line_number":308,"context_line":"        secgroups_to_delete \u003d resp_body[\u0027security_groups\u0027]"},{"line_number":309,"context_line":"        for secgroup in secgroups_to_delete:"}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_a84bda57","line":306,"in_reply_to":"baa201ad_f29fa401","updated":"2014-09-30 20:57:46.000000000","message":"Done","commit_id":"44d07ca0b107c9e360d80c59d1702d5e32821128"}]}