)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"5a6652caefd74bdc3fcb4e9340cee37bff12c1c0","unresolved":false,"context_lines":[{"line_number":10,"context_line":"filter out ESX hosts which are disconnected or in-maintenance mode"},{"line_number":11,"context_line":"while selecting datastores for creating volumes. Therefore, the"},{"line_number":12,"context_line":"check to filter out unusable ESX hosts before calling the method"},{"line_number":13,"context_line":"to select datastores is redundant. This patch removes the redundant"},{"line_number":14,"context_line":"check."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Partial-bug: #1556902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_d843bf40","line":13,"range":{"start_line":13,"start_character":21,"end_line":13,"end_character":33},"updated":"2016-10-13 13:56:33.000000000","message":"Patch seems good, just have a question. If this is redundant, where is the part its checking?","commit_id":"c4a634d2feba0fe97dae6e20f8dbf37b73d2d13d"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"7e82467ce3f23eb76d42d7f0943181cc971063b5","unresolved":false,"context_lines":[{"line_number":10,"context_line":"filter out ESX hosts which are disconnected or in-maintenance mode"},{"line_number":11,"context_line":"while selecting datastores for creating volumes. Therefore, the"},{"line_number":12,"context_line":"check to filter out unusable ESX hosts before calling the method"},{"line_number":13,"context_line":"to select datastores is redundant. This patch removes the redundant"},{"line_number":14,"context_line":"check."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Partial-bug: #1556902"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_231fed08","line":13,"range":{"start_line":13,"start_character":21,"end_line":13,"end_character":33},"in_reply_to":"fa6399be_d843bf40","updated":"2016-10-14 18:10:00.000000000","message":"It was added by the commit referred above.\n\nSee https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/vmware/datastore.py#L209","commit_id":"c4a634d2feba0fe97dae6e20f8dbf37b73d2d13d"}]}