)]}'
{"tempest/api/volume/test_volumes_clone.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3292064898a94d2227c82e42d49493c22a66e675","unresolved":false,"context_lines":[{"line_number":17,"context_line":"from tempest import config"},{"line_number":18,"context_line":"from tempest import test"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import testtools"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"CONF \u003d config.CONF"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_186a76b3","line":20,"updated":"2016-08-29 19:55:37.000000000","message":"This is a third party import, so should go in a section before the tempest imports.","commit_id":"1287ce0ecb7ab270f6ade7c8a3113825ea2b71d3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3292064898a94d2227c82e42d49493c22a66e675","unresolved":false,"context_lines":[{"line_number":25,"context_line":"class VolumesCloneTest(base.BaseVolumeTest):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    @test.idempotent_id(\u00279adae371-a257-43a5-9555-dc7c88e66e0e\u0027)"},{"line_number":28,"context_line":"    @testtools.skipUnless(CONF.volume_feature_enabled.clone,"},{"line_number":29,"context_line":"                          \u0027Cinder volume clones are disabled\u0027)"},{"line_number":30,"context_line":"    def test_create_from_volume(self):"},{"line_number":31,"context_line":"        # Creates a volume from another volume passing a size different from"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_78b46234","line":28,"updated":"2016-08-29 19:55:37.000000000","message":"Seems this should be in the skip_checks class method, that way if someone adds a new test case in this class we won\u0027t need the same decorator on that test case.","commit_id":"1287ce0ecb7ab270f6ade7c8a3113825ea2b71d3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"31db7f5958aad1e03a3c1195adf0f40ab505d4f3","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class VolumesCloneTest(base.BaseVolumeTest):"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    @test.idempotent_id(\u00279adae371-a257-43a5-9555-dc7c88e66e0e\u0027)"},{"line_number":27,"context_line":"    def test_create_from_volume(self):"},{"line_number":28,"context_line":"        # Creates a volume from another volume passing a size different from"},{"line_number":29,"context_line":"        # the source volume."}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_5a2372a7","side":"PARENT","line":26,"updated":"2016-08-31 22:51:09.000000000","message":"You also can\u0027t remove this.","commit_id":"3d8118bee9c9a5ccec91d4b047a93447db173131"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2597ec70d6a75b38bd0236a6e4593d61eba05b97","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import testtools"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest.api.volume import base"},{"line_number":19,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_6fceb984","line":16,"updated":"2016-08-31 21:11:59.000000000","message":"This isn\u0027t used.","commit_id":"afb7e3d9df70c9680608d447737ae08e5ff5e13d"}],"tempest/api/volume/test_volumes_clone_negative.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3292064898a94d2227c82e42d49493c22a66e675","unresolved":false,"context_lines":[{"line_number":26,"context_line":"class VolumesCloneTest(base.BaseVolumeTest):"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @test.idempotent_id(\u00279adae371-a257-43a5-459a-dc7c88e66e0e\u0027)"},{"line_number":29,"context_line":"    @testtools.skipUnless(CONF.volume_feature_enabled.clone,"},{"line_number":30,"context_line":"                          \u0027Cinder volume clones are disabled\u0027)"},{"line_number":31,"context_line":"    def test_create_from_volume_decreasing_size(self):"},{"line_number":32,"context_line":"        # Creates a volume from another volume passing a size different from"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_1898b6bf","line":29,"updated":"2016-08-29 19:55:37.000000000","message":"Same comment as the other module.","commit_id":"1287ce0ecb7ab270f6ade7c8a3113825ea2b71d3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"31db7f5958aad1e03a3c1195adf0f40ab505d4f3","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class VolumesCloneTest(base.BaseVolumeTest):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    @test.idempotent_id(\u00279adae371-a257-43a5-459a-dc7c88e66e0e\u0027)"},{"line_number":28,"context_line":"    def test_create_from_volume_decreasing_size(self):"},{"line_number":29,"context_line":"        # Creates a volume from another volume passing a size different from"},{"line_number":30,"context_line":"        # the source volume."}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_9a35fadc","side":"PARENT","line":27,"updated":"2016-08-31 22:51:09.000000000","message":"You can\u0027t remove this.","commit_id":"3d8118bee9c9a5ccec91d4b047a93447db173131"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2597ec70d6a75b38bd0236a6e4593d61eba05b97","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import testtools"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest.api.volume import base"},{"line_number":19,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_efa009a8","line":16,"updated":"2016-08-31 21:11:59.000000000","message":"ditto","commit_id":"afb7e3d9df70c9680608d447737ae08e5ff5e13d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2597ec70d6a75b38bd0236a6e4593d61eba05b97","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        if not CONF.volume_feature_enabled.clone:"},{"line_number":33,"context_line":"            raise cls.skipException(\"Cinder volume clones are disabled\") "},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def test_create_from_volume_decreasing_size(self):"},{"line_number":37,"context_line":"        # Creates a volume from another volume passing a size different from"},{"line_number":38,"context_line":"        # the source volume."}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_6f95d93e","line":35,"updated":"2016-08-31 21:11:59.000000000","message":"Seems you have one too many newlines here.","commit_id":"afb7e3d9df70c9680608d447737ae08e5ff5e13d"}]}
