)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"6bf39b0ae8371d6cdfb26cb3d43b0affafc03912","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix volume manage"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After moving the manage operation to support Active/Active the scheduler"},{"line_number":10,"context_line":"no longer coud find the backend to place managed volumes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"The issue comes from the fact that the user can request managing a"},{"line_number":13,"context_line":"volume just with the host, but if that backend is in a cluster the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a3c09a3_86afe528","line":10,"range":{"start_line":10,"start_character":10,"end_line":10,"end_character":14},"updated":"2017-01-18 12:37:30.000000000","message":"could","commit_id":"03ad26cdaf0a2e2fb0385ebadf85a4d42a785e8f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"24375d4675af1b3e1c55fcbc6e5ce4f7c7953100","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix volume manage"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After moving the manage operation to support Active/Active the scheduler"},{"line_number":10,"context_line":"no longer coud find the backend to place managed volumes."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"The issue comes from the fact that the user can request managing a"},{"line_number":13,"context_line":"volume just with the host, but if that backend is in a cluster the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a3c09a3_6b98d278","line":10,"range":{"start_line":10,"start_character":10,"end_line":10,"end_character":14},"in_reply_to":"7a3c09a3_86afe528","updated":"2017-01-18 14:19:30.000000000","message":"fixed in local branch, will get updated if I submit another patch.","commit_id":"03ad26cdaf0a2e2fb0385ebadf85a4d42a785e8f"}],"cinder/scheduler/manager.py":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"c117310835931f520ee9a583556cd7b9368f906d","unresolved":false,"context_lines":[{"line_number":314,"context_line":"            # At the API we didn\u0027t have the pool info, so the volume DB entry"},{"line_number":315,"context_line":"            # was created without it, now we add it."},{"line_number":316,"context_line":"            volume.host \u003d backend.host"},{"line_number":317,"context_line":"            volume.cluster_name \u003d backend.cluster_name"},{"line_number":318,"context_line":"            volume.save()"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"        except exception.NoValidBackend as ex:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_e045ca42","line":317,"range":{"start_line":317,"start_character":12,"end_line":317,"end_character":54},"updated":"2017-01-18 11:02:24.000000000","message":"It should be possible to determine service\u0027s cluster on API level, as host is passed into manage_existing. Why do we need to reset it here?","commit_id":"03ad26cdaf0a2e2fb0385ebadf85a4d42a785e8f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ced30e1e8aea4aee22ce65988961bb3fa99bea84","unresolved":false,"context_lines":[{"line_number":314,"context_line":"            # At the API we didn\u0027t have the pool info, so the volume DB entry"},{"line_number":315,"context_line":"            # was created without it, now we add it."},{"line_number":316,"context_line":"            volume.host \u003d backend.host"},{"line_number":317,"context_line":"            volume.cluster_name \u003d backend.cluster_name"},{"line_number":318,"context_line":"            volume.save()"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"        except exception.NoValidBackend as ex:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_6346447c","line":317,"range":{"start_line":317,"start_character":12,"end_line":317,"end_character":54},"in_reply_to":"7a3c09a3_e045ca42","updated":"2017-01-18 11:27:43.000000000","message":"The comment is valid for both the host as well as the cluster_name, because the cluster_name field should contain the pool and even when we are already setting the field in the API we couldn\u0027t set the pool right there.","commit_id":"03ad26cdaf0a2e2fb0385ebadf85a4d42a785e8f"}]}