)]}'
{"etc/tempest.conf.sample":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"c0fc5d00af08a7aeb12bc7c2e0313a4fdbb81518","unresolved":false,"context_lines":[{"line_number":1135,"context_line":"# Dict of volume type extra specs to use when creating a new"},{"line_number":1136,"context_line":"# volume type (dict value)"},{"line_number":1137,"context_line":"#volume_type_extra_specs\u003d"},{"line_number":1138,"context_line":""},{"line_number":1139,"context_line":""},{"line_number":1140,"context_line":"[volume-feature-enabled]"},{"line_number":1141,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa98f980_be109e50","line":1138,"updated":"2014-09-16 17:40:19.000000000","message":"Hi , \nSince its dictionary value and not int or string , it will be nice to see example","commit_id":"6d6594f7f305fbfaa9e03ba997a36203df707b57"}],"tempest/api/volume/admin/test_volume_types.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"97c7ae04365fa013c42d55d8bd55114ffff2dff9","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        vendor \u003d CONF.volume.vendor_name"},{"line_number":48,"context_line":"        extra_specs \u003d {\"storage_protocol\": proto,"},{"line_number":49,"context_line":"                       \"vendor_name\": vendor}"},{"line_number":50,"context_line":"        extra_specs.update(CONF.volume.volume_type_extra_specs)"},{"line_number":51,"context_line":"        body \u003d {}"},{"line_number":52,"context_line":"        _, body \u003d self.client.create_volume_type("},{"line_number":53,"context_line":"            vol_type_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_e18d307e","line":50,"updated":"2014-09-16 13:16:39.000000000","message":"Hi , \nThis value is optional  and other storage types are working without it .\nWhy this backend is different ?  IMO if it works on LVM and netapp iscsi it should pass on all backend .\nWhat do u think ?","commit_id":"6d6594f7f305fbfaa9e03ba997a36203df707b57"},{"author":{"_account_id":6509,"name":"Roman Bogorodskiy","email":"rbogorodskiy@mirantis.com","username":"novel"},"change_message_id":"e8bcd5e0367ad63e1591454c762833571e2bfd70","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        vendor \u003d CONF.volume.vendor_name"},{"line_number":48,"context_line":"        extra_specs \u003d {\"storage_protocol\": proto,"},{"line_number":49,"context_line":"                       \"vendor_name\": vendor}"},{"line_number":50,"context_line":"        extra_specs.update(CONF.volume.volume_type_extra_specs)"},{"line_number":51,"context_line":"        body \u003d {}"},{"line_number":52,"context_line":"        _, body \u003d self.client.create_volume_type("},{"line_number":53,"context_line":"            vol_type_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_63bdbdd7","line":50,"in_reply_to":"fa98f980_e18d307e","updated":"2014-09-16 17:22:17.000000000","message":"Benny,\n\nThis backend is different in not working without an explicit volume type because it stores the Coraid repository name as a volume type key. Without it it simply doesn\u0027t know what repository to use.\n\nUnfortunately, I have no information why it is done this way. Probably the reason was to allow using multiple repositories within a single instance of Cinder and multibackend support wasn\u0027t available at that time (that\u0027s just a guess, need to go through git log).\n\nI think it\u0027d be good to make the Coraid backend use config file instead and suggest users to use multi-backend if they want to use more than one Coraid repository.\n\nOn the other hand, that kind of change is probably large enough to fit Juno, but I guess CI still needs to be done as it\u0027s a requirement for backends now.","commit_id":"6d6594f7f305fbfaa9e03ba997a36203df707b57"}]}