)]}'
{"cinder/volume/drivers/hitachi/hnas_nfs.py":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"75b7caeba3d7f7a9871ff65321de51d3992404ba","unresolved":false,"context_lines":[{"line_number":284,"context_line":"        :param src_vref: reference to the source volume"},{"line_number":285,"context_line":"        :returns: the provider_location of the cloned volume"},{"line_number":286,"context_line":"        \"\"\""},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"        if volume.volume_type_id !\u003d src_vref.volume_type_id:"},{"line_number":289,"context_line":"            msg \u003d _(\"Source and cloned volumes should have the same \""},{"line_number":290,"context_line":"                    \"volume type. \")"},{"line_number":291,"context_line":"            LOG.error(msg)"},{"line_number":292,"context_line":"            raise exception.InvalidVolumeType(msg)"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"        vol_size \u003d volume.size"},{"line_number":295,"context_line":"        src_vol_size \u003d src_vref.size"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_0c8ea5cf","line":292,"range":{"start_line":287,"start_character":0,"end_line":292,"end_character":50},"updated":"2017-01-31 13:01:08.000000000","message":"-1: There are some situations where you could have different backends configured to the same pool, so, checking the volume type id wouldn\u0027t work in that case.\nYou need to extract the pool information for both volumes and check if they match.","commit_id":"700d6e731a2a3c7142e911fbab751dfdaa476d62"},{"author":{"_account_id":14865,"name":"Adriano Freires Rosso","email":"adriano.rosso@fit-tecnologia.org.br","username":"adriano_rosso"},"change_message_id":"926ffeab9aef772d80af64f803dccbb85eb5fa8c","unresolved":false,"context_lines":[{"line_number":284,"context_line":"        :param src_vref: reference to the source volume"},{"line_number":285,"context_line":"        :returns: the provider_location of the cloned volume"},{"line_number":286,"context_line":"        \"\"\""},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"        if volume.volume_type_id !\u003d src_vref.volume_type_id:"},{"line_number":289,"context_line":"            msg \u003d _(\"Source and cloned volumes should have the same \""},{"line_number":290,"context_line":"                    \"volume type. \")"},{"line_number":291,"context_line":"            LOG.error(msg)"},{"line_number":292,"context_line":"            raise exception.InvalidVolumeType(msg)"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"        vol_size \u003d volume.size"},{"line_number":295,"context_line":"        src_vol_size \u003d src_vref.size"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_3e6d35cc","line":292,"range":{"start_line":287,"start_character":0,"end_line":292,"end_character":50},"in_reply_to":"3a461143_0c8ea5cf","updated":"2017-01-31 18:25:49.000000000","message":"This configuration wouldn\u0027t make sense for this backend. It\u0027s not recommended to configure 2 HNAS backends to the same pool/export and, if it\u0027s done, they would be the exactly same thing.\nI\u0027m going to improve the log message to make things clearer.","commit_id":"700d6e731a2a3c7142e911fbab751dfdaa476d62"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"4c019d3d7f092bd248e30da208d81bbd9b1ae851","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        # clone operations."},{"line_number":291,"context_line":"        if volume.volume_type_id !\u003d src_vref.volume_type_id:"},{"line_number":292,"context_line":"            msg \u003d _(\"Source and cloned volumes should have the same \""},{"line_number":293,"context_line":"                    \"volume type since they will be created in the same \""},{"line_number":294,"context_line":"                    \"pool.\")"},{"line_number":295,"context_line":"            LOG.error(msg)"},{"line_number":296,"context_line":"            raise exception.InvalidVolumeType(msg)"},{"line_number":297,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_f621f79e","line":294,"range":{"start_line":293,"start_character":33,"end_line":294,"end_character":26},"updated":"2017-02-01 12:05:49.000000000","message":"I think this extra part is making the message confuse.","commit_id":"2067a01d34acd7480819aa4fd0512b2ac2e68cfd"},{"author":{"_account_id":14865,"name":"Adriano Freires Rosso","email":"adriano.rosso@fit-tecnologia.org.br","username":"adriano_rosso"},"change_message_id":"639afcb8399d734fff25e38cbb725d6881ac956f","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        # clone operations."},{"line_number":291,"context_line":"        if volume.volume_type_id !\u003d src_vref.volume_type_id:"},{"line_number":292,"context_line":"            msg \u003d _(\"Source and cloned volumes should have the same \""},{"line_number":293,"context_line":"                    \"volume type since they will be created in the same \""},{"line_number":294,"context_line":"                    \"pool.\")"},{"line_number":295,"context_line":"            LOG.error(msg)"},{"line_number":296,"context_line":"            raise exception.InvalidVolumeType(msg)"},{"line_number":297,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_b38db72c","line":294,"range":{"start_line":293,"start_character":33,"end_line":294,"end_character":26},"in_reply_to":"3a461143_f621f79e","updated":"2017-02-01 15:45:59.000000000","message":"Done","commit_id":"2067a01d34acd7480819aa4fd0512b2ac2e68cfd"}],"releasenotes/notes/fix-hnas-clone-with-different-volume-type-b969897cba2610cc.yaml":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"4c019d3d7f092bd248e30da208d81bbd9b1ae851","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - HNAS inconsistencies when cloning volumes using volume types"},{"line_number":4,"context_line":"    that are not the same volume type of the source volume."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a461143_a7c2f882","line":4,"range":{"start_line":2,"start_character":4,"end_line":4,"end_character":59},"updated":"2017-02-01 12:05:49.000000000","message":"Just to make clear and reflect what the patch is fixing:\n - Fixed HNAS bug that placed a cloned volume in the same pool as the source even if the clone had a different pool specification. That is not allowed and the driver will not permit this operation.","commit_id":"2067a01d34acd7480819aa4fd0512b2ac2e68cfd"},{"author":{"_account_id":14865,"name":"Adriano Freires Rosso","email":"adriano.rosso@fit-tecnologia.org.br","username":"adriano_rosso"},"change_message_id":"639afcb8399d734fff25e38cbb725d6881ac956f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - HNAS inconsistencies when cloning volumes using volume types"},{"line_number":4,"context_line":"    that are not the same volume type of the source volume."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a461143_d392434b","line":4,"range":{"start_line":2,"start_character":4,"end_line":4,"end_character":59},"in_reply_to":"3a461143_a7c2f882","updated":"2017-02-01 15:45:59.000000000","message":"Done","commit_id":"2067a01d34acd7480819aa4fd0512b2ac2e68cfd"}]}