)]}'
{"tempest/services/volume/json/snapshots_client.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"be0f65b597dc223ac7fcde9f5c8615e703c01fab","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2012 OpenStack, LLC"},{"line_number":4,"context_line":"# All Rights Reserved."},{"line_number":5,"context_line":"#"},{"line_number":6,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FM%2FI%3D","line":3,"updated":"2013-02-03 02:37:06.000000000","message":"I\u0027m pretty sure that you\u0027re not meant to assign copyright to OpenStack LLC anymore - at the very least hasn\u0027t it been replaced by the OpenStack foundation? And the year is wrong anyway :-)\n\nhttp://wiki.openstack.org/Governance/Proposed/Policy%20on%20Copyright%20and%20License%20headers","commit_id":"96fad65ebc72a239d17aee14142798a41f079c0f"}],"tempest/services/volume/xml/volumes_client.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"c465e6e9d87b5a2bbbcf1dfdb82cc00b3a887184","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        :param snapshot_id: When specified the volume is created from"},{"line_number":102,"context_line":"                            this snapshot"},{"line_number":103,"context_line":"        \"\"\""},{"line_number":104,"context_line":"        #NOTE(afazekas): it should use a volume namespace"},{"line_number":105,"context_line":"        volume \u003d Element(\"volume\", xmlns\u003dXMLNS_11, size\u003dsize)"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"        if \u0027metadata\u0027 in kwargs:"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAM3%2F%2FEWM%3D","line":104,"updated":"2013-02-08 04:38:15.000000000","message":"What does the above mean?","commit_id":"36b1fcf4171b8156a46d70448e4287bf36433909"}],"tempest/tests/volume/base.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"83db7aa6d4ba30c21888fd69861dba635afbfd1e","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"import nose"},{"line_number":22,"context_line":"import testtools"},{"line_number":23,"context_line":"import traceback"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from tempest import clients"},{"line_number":26,"context_line":"from tempest.common.utils.data_utils import rand_name"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FO3g%3D","line":23,"updated":"2013-02-01 00:38:21.000000000","message":"?","commit_id":"b8a558a0774ca27f8bfbf6e94e2fe2913eaf4d37"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"be0f65b597dc223ac7fcde9f5c8615e703c01fab","unresolved":false,"context_lines":[{"line_number":21,"context_line":"import nose"},{"line_number":22,"context_line":"import testtools"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from tempest import clients"},{"line_number":26,"context_line":"from tempest.common.utils.data_utils import rand_name"},{"line_number":27,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FM%2FE%3D","line":24,"updated":"2013-02-03 02:37:06.000000000","message":"Is this stray whitespace? I wonder if the pep8 checks are working correctly...","commit_id":"96fad65ebc72a239d17aee14142798a41f079c0f"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"be0f65b597dc223ac7fcde9f5c8615e703c01fab","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        for volume in cls.volumes:"},{"line_number":162,"context_line":"            try:"},{"line_number":163,"context_line":"                cls.servers_client.wait_for_resource_deletion(volume[\u0027id\u0027])"},{"line_number":164,"context_line":"            except Exception:"},{"line_number":165,"context_line":"                pass"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FM%2B8%3D","line":164,"updated":"2013-02-03 02:37:06.000000000","message":"Should we really be ignoring problems in cleanup? Similar for clear_snapshots. One issue I\u0027ve run into is sometimes a second run of tempest fails although the first run succeeded but some cleanup wasn\u0027t occurring properly which caused problems for the second run.","commit_id":"96fad65ebc72a239d17aee14142798a41f079c0f"}],"tempest/tests/volume/test_volumes_snapshots.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"d022bb3b42566eb5ab5f5cfe1dd64bb2c0ee6504","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        self.snapshots_client.wait_for_resource_deletion(snapshot[\u0027id\u0027])"},{"line_number":45,"context_line":"        self.client.delete_volume(volume_snap)"},{"line_number":46,"context_line":"        self.client.delete_volume(volume_origin)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"class VolumesSnapshotTestXML(base.BaseVolumeTestXML,"},{"line_number":50,"context_line":"                             VolumesSnapshotTestBase):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FPmo%3D","line":47,"updated":"2013-01-31 18:06:43.000000000","message":"We need to add a wait_for_resource_deletion on these delete_volume calls, else we may run into the situation we\u0027ve seen in the past with extra volumes laying around during next test.","commit_id":"923271d0d00edecc6f24ec1f4c4445e2756067a8"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"83db7aa6d4ba30c21888fd69861dba635afbfd1e","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        self.assertEqual(200, resp.status)  # 200 ???? 202 expected"},{"line_number":40,"context_line":"        self.assertTrue(\u0027id\u0027 in volume_snap)"},{"line_number":41,"context_line":"        self.client.wait_for_volume_status(volume_snap[\u0027id\u0027], \u0027available\u0027)"},{"line_number":42,"context_line":"        #TODO(afazekas): it may leak resources on error or fail"},{"line_number":43,"context_line":"        self.snapshots_client.delete_snapshot(snapshot[\u0027id\u0027])"},{"line_number":44,"context_line":"        self.snapshots_client.wait_for_resource_deletion(snapshot[\u0027id\u0027])"},{"line_number":45,"context_line":"        self.client.delete_volume(volume_snap[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FO3s%3D","line":42,"updated":"2013-02-01 00:38:21.000000000","message":"To make the tests more idempotent and robust, it would be nice to have this fixed before merging.","commit_id":"b8a558a0774ca27f8bfbf6e94e2fe2913eaf4d37"}]}