)]}'
{"tempest/tests/volume/admin/test_volume_types_extra_specs.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"bb4caa162708137e252ce24bab83aeedd19db97c","unresolved":false,"context_lines":[{"line_number":38,"context_line":"                                                               auth_url,"},{"line_number":39,"context_line":"                                                               adm_tenant)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        vol_type_name \u003d rand_name(\u0027Volume-type-\u0027)"},{"line_number":42,"context_line":"        cls.extra_spec \u003d {\"spec1\": \"val1\"}"},{"line_number":43,"context_line":"        resp, cls.volume_type \u003d cls.client.create_volume_type(vol_type_name)"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FBqs%3D","line":41,"updated":"2013-02-11 08:10:19.000000000","message":"looks like this  line not in use.\n\nThe same issue in the test_volume_types_extra_specs_negative.py.","commit_id":"31f382dd45fd934ed3a12e53e3f61c4defcc1737"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"029d0ef0883c3be918ece165df9263dd1748abda","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        vol_type_name \u003d rand_name(\u0027Volume-type-\u0027)"},{"line_number":42,"context_line":"        cls.extra_spec \u003d {\"spec1\": \"val1\"}"},{"line_number":43,"context_line":"        resp, cls.volume_type \u003d cls.client.create_volume_type(vol_type_name)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @classmethod"},{"line_number":46,"context_line":"    def tearDownClass(cls):"},{"line_number":47,"context_line":"        super(VolumeTypesExtraSpecsTest, cls).tearDownClass()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FBVs%3D","line":44,"updated":"2013-02-11 13:20:34.000000000","message":"Probably an extra-spec usage is missing from here.","commit_id":"31f382dd45fd934ed3a12e53e3f61c4defcc1737"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"d007bde6e9f93e878d4c71ab91285fa423a98cfc","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        vol_type_name \u003d rand_name(\u0027Volume-type-\u0027)"},{"line_number":42,"context_line":"        cls.extra_spec \u003d {\"spec1\": \"val1\"}"},{"line_number":43,"context_line":"        resp, cls.volume_type \u003d cls.client.create_volume_type(vol_type_name)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @classmethod"},{"line_number":46,"context_line":"    def tearDownClass(cls):"},{"line_number":47,"context_line":"        super(VolumeTypesExtraSpecsTest, cls).tearDownClass()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2B9tk%3D","line":44,"in_reply_to":"AAAAM3%2F%2FBVs%3D","updated":"2013-02-13 03:08:40.000000000","message":"I think that probably it used to be used as a common setting for all the tests. But that was changed at some point for test specific settings.","commit_id":"31f382dd45fd934ed3a12e53e3f61c4defcc1737"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"bb4caa162708137e252ce24bab83aeedd19db97c","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    def test_volume_type_extra_specs_update(self):"},{"line_number":69,"context_line":"        # Update volume type extra specs"},{"line_number":70,"context_line":"        try:"},{"line_number":71,"context_line":"            extra_spec \u003d {\"spec1\": \"val2\"}"},{"line_number":72,"context_line":"            resp, body \u003d self.client.\\"},{"line_number":73,"context_line":"            update_volume_type_extra_specs(self.volume_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FBqU%3D","line":70,"updated":"2013-02-11 08:10:19.000000000","message":"Is it possible to have all of these to have different key / value ?","commit_id":"31f382dd45fd934ed3a12e53e3f61c4defcc1737"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"029d0ef0883c3be918ece165df9263dd1748abda","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        try:"},{"line_number":71,"context_line":"            extra_spec \u003d {\"spec1\": \"val2\"}"},{"line_number":72,"context_line":"            resp, body \u003d self.client.\\"},{"line_number":73,"context_line":"            update_volume_type_extra_specs(self.volume_type[\u0027id\u0027],"},{"line_number":74,"context_line":"                                           extra_spec.keys()[0],"},{"line_number":75,"context_line":"                                           extra_spec)"},{"line_number":76,"context_line":"            self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FBV4%3D","line":73,"updated":"2013-02-11 13:20:34.000000000","message":"It is not really an update!","commit_id":"31f382dd45fd934ed3a12e53e3f61c4defcc1737"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"d007bde6e9f93e878d4c71ab91285fa423a98cfc","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        try:"},{"line_number":71,"context_line":"            extra_spec \u003d {\"spec1\": \"val2\"}"},{"line_number":72,"context_line":"            resp, body \u003d self.client.\\"},{"line_number":73,"context_line":"            update_volume_type_extra_specs(self.volume_type[\u0027id\u0027],"},{"line_number":74,"context_line":"                                           extra_spec.keys()[0],"},{"line_number":75,"context_line":"                                           extra_spec)"},{"line_number":76,"context_line":"            self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2B9tU%3D","line":73,"in_reply_to":"AAAAM3%2F%2FBV4%3D","updated":"2013-02-13 03:08:40.000000000","message":"Good point. This is another example of a test running after another test","commit_id":"31f382dd45fd934ed3a12e53e3f61c4defcc1737"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"8fd80229f22f3fd39771358f53a87da2fbb7ceb6","unresolved":false,"context_lines":[{"line_number":106,"context_line":"    def test_volume_type_extra_spec_create_get(self):"},{"line_number":107,"context_line":"        # Create/get volume type extra spec"},{"line_number":108,"context_line":"        try:"},{"line_number":109,"context_line":"            extra_specs \u003d {\"spec4\": \"val1\"}"},{"line_number":110,"context_line":"            resp, body \u003d self.client.\\"},{"line_number":111,"context_line":"            create_volume_type_extra_specs(self.volume_type[\u0027id\u0027], extra_specs)"},{"line_number":112,"context_line":"            self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2B62k%3D","line":109,"updated":"2013-02-14 14:39:33.000000000","message":"Couldn\u0027t we combine create_delete and create_get tests into a single create_get_delete test?","commit_id":"b3cb8e168921fed4f916b621aa0cb9873c705067"}]}