)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"eebe8b5e326034ab6e3ee1c79b5af96e8afca4e7","unresolved":false,"context_lines":[{"line_number":13,"context_line":"NfsDriver class is inherited by multiple vendor drivers. Affected"},{"line_number":14,"context_line":"drivers are listed below:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"* NetAppNfsDriver"},{"line_number":17,"context_line":"* NetAppCmodeNfsDriver"},{"line_number":18,"context_line":"* NetApp7modeNfsDriver"},{"line_number":19,"context_line":"* CohoDriver"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a430d35_6057f826","line":16,"range":{"start_line":16,"start_character":2,"end_line":16,"end_character":17},"updated":"2017-02-09 17:23:38.000000000","message":"I believe this one is only the base class for the 2 bellow.","commit_id":"7631f8acd37f3a5161da08d82430e94527a2a1ca"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"7bdbb658ed9422c6db8763b250ad9275a1b02b7e","unresolved":false,"context_lines":[{"line_number":13,"context_line":"NfsDriver class is inherited by multiple vendor drivers. Affected"},{"line_number":14,"context_line":"drivers are listed below:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"* NetAppNfsDriver"},{"line_number":17,"context_line":"* NetAppCmodeNfsDriver"},{"line_number":18,"context_line":"* NetApp7modeNfsDriver"},{"line_number":19,"context_line":"* CohoDriver"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa31d9ce_b82462af","line":16,"range":{"start_line":16,"start_character":2,"end_line":16,"end_character":17},"in_reply_to":"1a430d35_6057f826","updated":"2017-02-13 12:43:02.000000000","message":"True.","commit_id":"7631f8acd37f3a5161da08d82430e94527a2a1ca"}],"cinder/volume/drivers/nfs.py":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"eebe8b5e326034ab6e3ee1c79b5af96e8afca4e7","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        return super(NfsDriver, self).create_volume(volume)"},{"line_number":507,"context_line":""},{"line_number":508,"context_line":"    @locked_volume_id_operation"},{"line_number":509,"context_line":"    def delete_volume(self, volume):"},{"line_number":510,"context_line":"        \"\"\"Deletes a logical volume.\"\"\""},{"line_number":511,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_f7e64baf","side":"PARENT","line":508,"range":{"start_line":508,"start_character":5,"end_line":508,"end_character":31},"updated":"2017-02-09 17:23:38.000000000","message":"What if we change this decorator (remofs.locked_volume_id_operation()) and add this behavior to all remotefs drivers?","commit_id":"25a37650c6fd7d6f0a5786448f417ee47fc88c13"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"7bdbb658ed9422c6db8763b250ad9275a1b02b7e","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        return super(NfsDriver, self).create_volume(volume)"},{"line_number":507,"context_line":""},{"line_number":508,"context_line":"    @locked_volume_id_operation"},{"line_number":509,"context_line":"    def delete_volume(self, volume):"},{"line_number":510,"context_line":"        \"\"\"Deletes a logical volume.\"\"\""},{"line_number":511,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa31d9ce_3873124f","side":"PARENT","line":508,"range":{"start_line":508,"start_character":5,"end_line":508,"end_character":31},"in_reply_to":"1a430d35_f7e64baf","updated":"2017-02-13 12:43:02.000000000","message":"I\u0027m trying to limit affected drivers - modifying this decorator directly would mean that this patch would affect a few more:\n\n* SmbfsDriver\n* VzStorage\n* windows.SmbfsDriver\n\nThat\u0027s why I\u0027m changing this directly.","commit_id":"25a37650c6fd7d6f0a5786448f417ee47fc88c13"},{"author":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"change_message_id":"a25b31d775f88d8421c705b8e7d831354f2c7dd2","unresolved":false,"context_lines":[{"line_number":499,"context_line":""},{"line_number":500,"context_line":"        self._stats \u003d data"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"    @coordination.synchronized(\u0027{self.driver_prefix}-{volume[id]}\u0027)"},{"line_number":503,"context_line":"    def create_volume(self, volume):"},{"line_number":504,"context_line":"        \"\"\"Apply locking to the create volume operation.\"\"\""},{"line_number":505,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba2be162_ea7ea068","line":502,"range":{"start_line":502,"start_character":54,"end_line":502,"end_character":64},"updated":"2017-03-03 22:08:53.000000000","message":"Why not volume.id? It\u0027s used that way in locked_volume_id_operation.","commit_id":"6ef6c190c350ac7cd8f7d39fde285a3ba09bf503"},{"author":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"change_message_id":"a25b31d775f88d8421c705b8e7d831354f2c7dd2","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        return super(NfsDriver, self).create_volume(volume)"},{"line_number":507,"context_line":""},{"line_number":508,"context_line":"    @coordination.synchronized(\u0027{self.driver_prefix}-{volume[id]}\u0027)"},{"line_number":509,"context_line":"    def delete_volume(self, volume):"},{"line_number":510,"context_line":"        \"\"\"Deletes a logical volume.\"\"\""},{"line_number":511,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba2be162_eaf5c0ec","line":508,"range":{"start_line":508,"start_character":54,"end_line":508,"end_character":64},"updated":"2017-03-03 22:08:53.000000000","message":"ditto","commit_id":"6ef6c190c350ac7cd8f7d39fde285a3ba09bf503"}]}