)]}'
{"tempest/scenario/test_keystoneclient.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"4136ed6941512165eeb6b6f9e41ce371b1efa5ff","unresolved":false,"context_lines":[{"line_number":17,"context_line":"from tempest.test import services"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"DEFAULT_DOMAIN_ID \u003d \u0027default\u0027"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class DefaultFixtures(object):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9nqM%3D","line":20,"updated":"2013-12-09 16:03:27.000000000","message":"this needs to come from the config, not be a string constant. As this needs to be runnable on any live cloud.","commit_id":"d5259f91936c7f671a238b8e8586a6f9a6eb46cc"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"4136ed6941512165eeb6b6f9e41ce371b1efa5ff","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class DefaultFixtures(object):"},{"line_number":24,"context_line":"    USERS \u003d ["},{"line_number":25,"context_line":"        {"},{"line_number":26,"context_line":"            \u0027id\u0027: \u0027foo\u0027,"},{"line_number":27,"context_line":"            \u0027name\u0027: \u0027FOO\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9nqI%3D","line":24,"updated":"2013-12-09 16:03:27.000000000","message":"we shouldn\u0027t be using static fixtures like this, these should be generated as random data, or else come from a config.","commit_id":"d5259f91936c7f671a238b8e8586a6f9a6eb46cc"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"4136ed6941512165eeb6b6f9e41ce371b1efa5ff","unresolved":false,"context_lines":[{"line_number":76,"context_line":"                              password\u003dself.config.identity.admin_password,"},{"line_number":77,"context_line":"                              tenant_name\u003dself.config.identity.tenant_name)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        project_name \u003d \u0027BAR\u0027"},{"line_number":80,"context_line":"        tenant \u003d client.tenants.create(project_name, description\u003d\u0027description\u0027,"},{"line_number":81,"context_line":"                                       enabled\u003dTrue)"},{"line_number":82,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9nqE%3D","line":79,"updated":"2013-12-09 16:03:27.000000000","message":"should be rand generated","commit_id":"d5259f91936c7f671a238b8e8586a6f9a6eb46cc"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"4136ed6941512165eeb6b6f9e41ce371b1efa5ff","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        return \"http://localhost:%s/v2.0\" % public_port"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    def _client(self, admin\u003dFalse, **kwargs):"},{"line_number":121,"context_line":"        from keystoneclient.v2_0 import client as ks_client"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        url \u003d self._admin_url() if admin else self._public_url()"},{"line_number":124,"context_line":"        kc \u003d ks_client.Client(endpoint\u003durl,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9npg%3D","line":121,"updated":"2013-12-09 16:03:27.000000000","message":"access to the client is through tempest/scenario/manager.py, you shouldn\u0027t be instantianting it directly","commit_id":"d5259f91936c7f671a238b8e8586a6f9a6eb46cc"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"4136ed6941512165eeb6b6f9e41ce371b1efa5ff","unresolved":false,"context_lines":[{"line_number":151,"context_line":"        self.assertEqual(tenants[0].id, self.tenant_bar[\u0027id\u0027])"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"class Kc11TestCase(CompatTestCase, KeystoneClientTests):"},{"line_number":155,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9noM%3D","line":154,"updated":"2013-12-09 16:03:27.000000000","message":"please no dead code","commit_id":"d5259f91936c7f671a238b8e8586a6f9a6eb46cc"}]}