)]}'
{"neutronclient/neutron/v2_0/auto_allocated_topology.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"1dadd8a79e9c85fedee14c29dce58864e3e1175f","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        client \u003d self.get_client()"},{"line_number":108,"context_line":"        tenant_id \u003d parsed_args.tenant_id or parsed_args.pos_tenant_id"},{"line_number":109,"context_line":"        client.delete_auto_allocated_topology(tenant_id)"},{"line_number":110,"context_line":"        print(_(\u0027Deleted topology for tenant %(tenant_id)s.\u0027) %"},{"line_number":111,"context_line":"              {\u0027tenant_id\u0027: tenant_id},"},{"line_number":112,"context_line":"              file\u003dself.app.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_d71ce36b","line":111,"range":{"start_line":110,"start_character":44,"end_line":111,"end_character":38},"updated":"2016-08-24 11:12:45.000000000","message":"nit: We usually use %s when only one substitution is used.","commit_id":"939189e9c2a1fce0a4b7c9c5685b49431f3451db"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cb0e72679ab076d0284392efe1d5f40524620115","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        client \u003d self.get_client()"},{"line_number":108,"context_line":"        tenant_id \u003d parsed_args.tenant_id or parsed_args.pos_tenant_id"},{"line_number":109,"context_line":"        client.delete_auto_allocated_topology(tenant_id)"},{"line_number":110,"context_line":"        print(_(\u0027Deleted topology for tenant %(tenant_id)s.\u0027) %"},{"line_number":111,"context_line":"              {\u0027tenant_id\u0027: tenant_id},"},{"line_number":112,"context_line":"              file\u003dself.app.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_8ff86b4b","line":111,"range":{"start_line":110,"start_character":44,"end_line":111,"end_character":38},"in_reply_to":"1ac06dbe_d71ce36b","updated":"2016-08-24 17:08:14.000000000","message":"ack","commit_id":"939189e9c2a1fce0a4b7c9c5685b49431f3451db"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"27161ac294b40ff9a4215ecb7ee4d83f258aec6a","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        tenant_id \u003d parsed_args.tenant_id or parsed_args.pos_tenant_id"},{"line_number":109,"context_line":"        client.delete_auto_allocated_topology(tenant_id)"},{"line_number":110,"context_line":"        # It tenant is None, let\u0027s be clear on what it means."},{"line_number":111,"context_line":"        tenant_id \u003d tenant_id or \u0027None (i.e. yourself)\u0027"},{"line_number":112,"context_line":"        print(_(\u0027Deleted topology for tenant %s.\u0027) % tenant_id,"},{"line_number":113,"context_line":"              file\u003dself.app.stdout)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_e849e2d1","line":111,"range":{"start_line":111,"start_character":45,"end_line":111,"end_character":53},"updated":"2016-08-25 10:38:57.000000000","message":"A tenant is not a person. (This is one reason why keystone renamed tenant to project.)\n\n  if tenant_id is None:\n      print(_(\"Deleted topology for your tenant.\") ...\n  else:\n      print(_(\u0027Deleted topology for tenant %s.\u0027) ...","commit_id":"3184a6d5254f73058f4a9e22e3036318968a8f51"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"67a566dc11e9048db4df53318e391feeb4dddcb3","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        tenant_id \u003d parsed_args.tenant_id or parsed_args.pos_tenant_id"},{"line_number":109,"context_line":"        client.delete_auto_allocated_topology(tenant_id)"},{"line_number":110,"context_line":"        # It tenant is None, let\u0027s be clear on what it means."},{"line_number":111,"context_line":"        tenant_id \u003d tenant_id or \u0027None (i.e. yourself)\u0027"},{"line_number":112,"context_line":"        print(_(\u0027Deleted topology for tenant %s.\u0027) % tenant_id,"},{"line_number":113,"context_line":"              file\u003dself.app.stdout)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_7898091c","line":111,"range":{"start_line":111,"start_character":45,"end_line":111,"end_character":53},"in_reply_to":"1ac06dbe_e849e2d1","updated":"2016-08-25 15:44:56.000000000","message":"can\u0027t do that, the tests barf at me and I really don\u0027t want to hijack my way through the test utils. Yourself doesn\u0027t necessarily refer to persons, does it? I thought it can refer to entities, whether they are objects, animals or tenants :)","commit_id":"3184a6d5254f73058f4a9e22e3036318968a8f51"}],"neutronclient/v2_0/client.py":[{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"e21bb001570773e3be8ea725273513d8b9b4c3ce","unresolved":false,"context_lines":[{"line_number":1810,"context_line":"    @debtcollector.renames.renamed_kwarg("},{"line_number":1811,"context_line":"        \u0027tenant_id\u0027, \u0027project_id\u0027, replace\u003dTrue)"},{"line_number":1812,"context_line":"    def delete_auto_allocated_topology(self, project_id, **_params):"},{"line_number":1813,"context_line":"        \"\"\"Delete information about a project\u0027s auto-allocated topology.\"\"\""},{"line_number":1814,"context_line":"        return self.delete("},{"line_number":1815,"context_line":"            self.auto_allocated_topology_path % project_id,"},{"line_number":1816,"context_line":"            params\u003d_params)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_914f1633","line":1813,"range":{"start_line":1813,"start_character":18,"end_line":1813,"end_character":35},"updated":"2016-08-24 08:03:03.000000000","message":"Nit: It deletes the topology, not just the information about it.","commit_id":"939189e9c2a1fce0a4b7c9c5685b49431f3451db"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"1dadd8a79e9c85fedee14c29dce58864e3e1175f","unresolved":false,"context_lines":[{"line_number":1810,"context_line":"    @debtcollector.renames.renamed_kwarg("},{"line_number":1811,"context_line":"        \u0027tenant_id\u0027, \u0027project_id\u0027, replace\u003dTrue)"},{"line_number":1812,"context_line":"    def delete_auto_allocated_topology(self, project_id, **_params):"},{"line_number":1813,"context_line":"        \"\"\"Delete information about a project\u0027s auto-allocated topology.\"\"\""},{"line_number":1814,"context_line":"        return self.delete("},{"line_number":1815,"context_line":"            self.auto_allocated_topology_path % project_id,"},{"line_number":1816,"context_line":"            params\u003d_params)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_e9ac96a0","line":1813,"range":{"start_line":1813,"start_character":18,"end_line":1813,"end_character":35},"in_reply_to":"1ac06dbe_914f1633","updated":"2016-08-24 11:12:45.000000000","message":"Agree","commit_id":"939189e9c2a1fce0a4b7c9c5685b49431f3451db"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cb0e72679ab076d0284392efe1d5f40524620115","unresolved":false,"context_lines":[{"line_number":1810,"context_line":"    @debtcollector.renames.renamed_kwarg("},{"line_number":1811,"context_line":"        \u0027tenant_id\u0027, \u0027project_id\u0027, replace\u003dTrue)"},{"line_number":1812,"context_line":"    def delete_auto_allocated_topology(self, project_id, **_params):"},{"line_number":1813,"context_line":"        \"\"\"Delete information about a project\u0027s auto-allocated topology.\"\"\""},{"line_number":1814,"context_line":"        return self.delete("},{"line_number":1815,"context_line":"            self.auto_allocated_topology_path % project_id,"},{"line_number":1816,"context_line":"            params\u003d_params)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_0fa3db50","line":1813,"range":{"start_line":1813,"start_character":18,"end_line":1813,"end_character":35},"in_reply_to":"1ac06dbe_e9ac96a0","updated":"2016-08-24 17:08:14.000000000","message":"ack","commit_id":"939189e9c2a1fce0a4b7c9c5685b49431f3451db"}]}