)]}'
{"cinder/volume/drivers/remotefs.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0da8a34719004ed8cbd6266692d1fbdcab4b7df8","unresolved":false,"context_lines":[{"line_number":1659,"context_line":""},{"line_number":1660,"context_line":""},{"line_number":1661,"context_line":"class RemoteFSSnapDriverDistributed(RemoteFSSnapDriverBase):"},{"line_number":1662,"context_line":"    @coordination.synchronized(\u0027{snapshot.volume.id}\u0027)"},{"line_number":1663,"context_line":"    def create_snapshot(self, snapshot):"},{"line_number":1664,"context_line":"        \"\"\"Apply locking to the create snapshot operation.\"\"\""},{"line_number":1665,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_af0cb293","line":1662,"updated":"2018-07-23 16:38:49.000000000","message":"This will no longer lock against extend_volume below.  Is this really safe for all drivers?","commit_id":"62cf90bbed2db66bb852b607e5de23819df0bb99"},{"author":{"_account_id":13915,"name":"Silvan Kaiser","email":"silvan@quobyte.com","username":"kaisers"},"change_message_id":"6e61de4690b81bebe2d286c38011c323edd0078f","unresolved":false,"context_lines":[{"line_number":1659,"context_line":""},{"line_number":1660,"context_line":""},{"line_number":1661,"context_line":"class RemoteFSSnapDriverDistributed(RemoteFSSnapDriverBase):"},{"line_number":1662,"context_line":"    @coordination.synchronized(\u0027{snapshot.volume.id}\u0027)"},{"line_number":1663,"context_line":"    def create_snapshot(self, snapshot):"},{"line_number":1664,"context_line":"        \"\"\"Apply locking to the create snapshot operation.\"\"\""},{"line_number":1665,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_8509892a","line":1662,"in_reply_to":"5f7c97a3_af0cb293","updated":"2018-07-24 06:28:37.000000000","message":"you\u0027re correct, this needs more though on a broader level.","commit_id":"62cf90bbed2db66bb852b607e5de23819df0bb99"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"54823bb397e8ba2ce673c9b4c652a26258a9fb7d","unresolved":false,"context_lines":[{"line_number":1688,"context_line":"        return self._copy_volume_to_image(context, volume, image_service,"},{"line_number":1689,"context_line":"                                          image_meta)"},{"line_number":1690,"context_line":""},{"line_number":1691,"context_line":"    @coordination.synchronized(\u0027{self.driver_prefix}-{volume.id}\u0027)"},{"line_number":1692,"context_line":"    def extend_volume(self, volume, size_gb):"},{"line_number":1693,"context_line":"        return self._extend_volume(volume, size_gb)"},{"line_number":1694,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_4ffdde3a","line":1691,"range":{"start_line":1691,"start_character":32,"end_line":1691,"end_character":53},"updated":"2018-07-23 16:45:27.000000000","message":"Good catch, some of these other operations should probably also be updated to lock just on the ID.","commit_id":"62cf90bbed2db66bb852b607e5de23819df0bb99"}]}