)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23113,"name":"Hariprasad S Venkatasubramanian","email":"hari.venkat@hpe.com","username":"hari.venkat"},"change_message_id":"e07dd710506d3f6e78792d891c49909830b3f846","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Merge Of:   0674742a (Merge \"[api-ref]Fix the volume metadata description error\")"},{"line_number":2,"context_line":"            3ebee7f6 ([api-ref]Add volumes/summary API doc)"},{"line_number":3,"context_line":"Author:     Jenkins \u003cjenkins@review.openstack.org\u003e"},{"line_number":4,"context_line":"AuthorDate: 2017-02-05 22:26:59 +0000"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_41f52655","side":"PARENT","line":1,"updated":"2017-02-07 06:07:35.000000000","message":"nit: Lefthand --\u003e HPE StoreVirtual","commit_id":"99efbaef1e805dbe695a7af808a068fdf392db1c"},{"author":{"_account_id":21406,"name":"Vijay Ladani","email":"vijay.ladani@hpe.com","username":"vladani"},"change_message_id":"6c64ddff86cc7524ceddf7c655a4544a22901ec6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Merge Of:   0674742a (Merge \"[api-ref]Fix the volume metadata description error\")"},{"line_number":2,"context_line":"            3ebee7f6 ([api-ref]Add volumes/summary API doc)"},{"line_number":3,"context_line":"Author:     Jenkins \u003cjenkins@review.openstack.org\u003e"},{"line_number":4,"context_line":"AuthorDate: 2017-02-05 22:26:59 +0000"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_a6f86ac4","side":"PARENT","line":1,"in_reply_to":"1a430d35_41f52655","updated":"2017-02-08 08:25:07.000000000","message":"Done","commit_id":"99efbaef1e805dbe695a7af808a068fdf392db1c"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"cdf677eedfa886a92e79734c74b432c2b8366f2d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Lefthand: create and restore volume backup fails"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When a user wants to perform parallel cinder backup volume"},{"line_number":10,"context_line":"creation and restoration operation, 3par driver ends the"},{"line_number":11,"context_line":"execution with SERVER_ALREADY_EXISTS error."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"While creating/restoring volume backups concurrently on HPE"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_d7e52638","line":10,"range":{"start_line":10,"start_character":35,"end_line":10,"end_character":42},"updated":"2017-02-08 05:13:19.000000000","message":"Please use proper driver name","commit_id":"ac45040dc5ffbe6933c10f2585b058d43ff7fa42"},{"author":{"_account_id":21406,"name":"Vijay Ladani","email":"vijay.ladani@hpe.com","username":"vladani"},"change_message_id":"6c64ddff86cc7524ceddf7c655a4544a22901ec6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Lefthand: create and restore volume backup fails"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When a user wants to perform parallel cinder backup volume"},{"line_number":10,"context_line":"creation and restoration operation, 3par driver ends the"},{"line_number":11,"context_line":"execution with SERVER_ALREADY_EXISTS error."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"While creating/restoring volume backups concurrently on HPE"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_c601f6b0","line":10,"range":{"start_line":10,"start_character":35,"end_line":10,"end_character":42},"in_reply_to":"1a430d35_d7e52638","updated":"2017-02-08 08:25:07.000000000","message":"Done","commit_id":"ac45040dc5ffbe6933c10f2585b058d43ff7fa42"}],"cinder/volume/drivers/hpe/hpe_lefthand_iscsi.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"c610209c5df676048e37ca0d04372cad0c46135d","unresolved":false,"context_lines":[{"line_number":713,"context_line":"        self.device_stats \u003d data"},{"line_number":714,"context_line":""},{"line_number":715,"context_line":"    @cinder_utils.trace"},{"line_number":716,"context_line":"    @cinder_utils.synchronized(\u0027lefthand\u0027, external\u003dTrue)"},{"line_number":717,"context_line":"    def initialize_connection(self, volume, connector):"},{"line_number":718,"context_line":"        \"\"\"Assigns the volume to a server."},{"line_number":719,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_a975ce33","line":716,"updated":"2017-06-06 16:35:33.000000000","message":"I would prefer to use the coordinated locks for this instead of the local file locks.","commit_id":"b2270bb4505a764262f28b20d167364bb102cfa4"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8b09a887e07df9b3a5b979bb70f24c1ebab03b09","unresolved":false,"context_lines":[{"line_number":713,"context_line":"        self.device_stats \u003d data"},{"line_number":714,"context_line":""},{"line_number":715,"context_line":"    @cinder_utils.trace"},{"line_number":716,"context_line":"    @cinder_utils.synchronized(\u0027lefthand\u0027, external\u003dTrue)"},{"line_number":717,"context_line":"    def initialize_connection(self, volume, connector):"},{"line_number":718,"context_line":"        \"\"\"Assigns the volume to a server."},{"line_number":719,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f201791_ccc7af12","line":716,"in_reply_to":"df140735_a975ce33","updated":"2017-06-22 10:42:45.000000000","message":"I totally agree with Walter, this needs to use the cinder.coordination.synchronized decorator instead.  For normal deployments (no DLM is configured) it will be a file lock anyway, so we are good.","commit_id":"b2270bb4505a764262f28b20d167364bb102cfa4"}]}