)]}'
{"tempest/api/volume/admin/test_volume_types.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"0e31538294c0d24e5b8ce445a699cd0ad63f405d","unresolved":false,"context_lines":[{"line_number":207,"context_line":"    @testtools.skipIf(CONF.volume_feature_enabled.api_v1,"},{"line_number":208,"context_line":"                      \u0027Cinder type_access_add does not support V1\u0027)"},{"line_number":209,"context_line":"    def test_volume_type_access_add(self):"},{"line_number":210,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_b87056a5","line":210,"range":{"start_line":210,"start_character":8,"end_line":210,"end_character":12},"updated":"2016-05-22 14:25:30.000000000","message":"Hi Liron , \nMaybe its better to create a class for type_access to inherit from v2 only .\n\nclass VolumeTypesAccessV2Test:(base.BaseVolumeAdminTest)\n def test_volume_type_access_add(self):\n................\n\nWe wont have the ignore message in the logs  , since its not a bug but not supported we should not test it at all and do not access to method that never will run ... \n\nBenny","commit_id":"ffd3f41a623c1d70cf46f143322d8b8acf4eda9c"}]}