)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"e961df5beb5b10bdab4a28fe2f2dc4e5bb02881c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix delete_snapshot error case in Pure driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Our previous error handling was incorrect for delete_snapshot. This"},{"line_number":10,"context_line":"adds a check on repsonse and correctly logs only a warning if"},{"line_number":11,"context_line":"required."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic73d92eaa5869efe1dc0e56d3156edd7dbcac2e8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_7a98706a","line":10,"updated":"2015-08-31 15:13:44.000000000","message":"repsonse -\u003e response","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"}],"cinder/volume/drivers/pure.py":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"03cbda3316c84744e15c20a3291b041c898ffd82","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                    # Happens if the snapshot does not exist."},{"line_number":207,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":208,"context_line":"                    LOG.warning(_LW(\"Snapshot deletion failed with \""},{"line_number":209,"context_line":"                                    \"message: %s\"), err.text)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def ensure_export(self, context, volume):"},{"line_number":212,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_ee851007","line":209,"updated":"2015-08-28 02:32:12.000000000","message":"Do you need a period on this one?","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"c832ff35c6528c0d584c41840fa9d95f654ca5bc","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                    # Happens if the snapshot does not exist."},{"line_number":207,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":208,"context_line":"                    LOG.warning(_LW(\"Snapshot deletion failed with \""},{"line_number":209,"context_line":"                                    \"message: %s\"), err.text)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def ensure_export(self, context, volume):"},{"line_number":212,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_51e79cab","line":209,"in_reply_to":"da20952f_4fdf310d","updated":"2015-08-31 12:37:44.000000000","message":"I guess you can add a quote to the message \u0027%s\u0027. Then append an period. So that the message can be more accurate and double periods won\u0027t be confusing.","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"2680a91830942fae2a70904ba98649a153e0a461","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                    # Happens if the snapshot does not exist."},{"line_number":207,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":208,"context_line":"                    LOG.warning(_LW(\"Snapshot deletion failed with \""},{"line_number":209,"context_line":"                                    \"message: %s\"), err.text)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def ensure_export(self, context, volume):"},{"line_number":212,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a7e1126_7a7aec78","line":209,"in_reply_to":"da20952f_51e79cab","updated":"2015-12-19 00:38:43.000000000","message":"To settle this, I\u0027m not going to add an extra period to this log message. It is correct as-is.","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"e36f5102b76e652cdbad6ce1e7ee6df34e177c11","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                    # Happens if the snapshot does not exist."},{"line_number":207,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":208,"context_line":"                    LOG.warning(_LW(\"Snapshot deletion failed with \""},{"line_number":209,"context_line":"                                    \"message: %s\"), err.text)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def ensure_export(self, context, volume):"},{"line_number":212,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_4fdf310d","line":209,"in_reply_to":"da20952f_69f0f5fe","updated":"2015-08-31 11:41:19.000000000","message":"I had a dim memory of something like this with your impure web api messages so that\u0027s why I asked instead of pouncing on you with a -1 :-)","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"f2732f0e2f061c99d17c7b031e67b0373360f569","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                    # Happens if the snapshot does not exist."},{"line_number":207,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":208,"context_line":"                    LOG.warning(_LW(\"Snapshot deletion failed with \""},{"line_number":209,"context_line":"                                    \"message: %s\"), err.text)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def ensure_export(self, context, volume):"},{"line_number":212,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_fa2613cd","line":209,"in_reply_to":"da20952f_ee851007","updated":"2015-08-28 13:45:10.000000000","message":"Yes.","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"},{"author":{"_account_id":13868,"name":"Danny Wilson","email":"daniel.wilson@purestorage.com","username":"dannywilson"},"change_message_id":"c1c38434749e7d799cc68805397ec04f1d56e1d7","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                    # Happens if the snapshot does not exist."},{"line_number":207,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":208,"context_line":"                    LOG.warning(_LW(\"Snapshot deletion failed with \""},{"line_number":209,"context_line":"                                    \"message: %s\"), err.text)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def ensure_export(self, context, volume):"},{"line_number":212,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_69f0f5fe","line":209,"in_reply_to":"da20952f_fa2613cd","updated":"2015-08-28 16:52:36.000000000","message":"This is a weird case.  When we get error messages from our web api they include periods.  In this case if I add a period here it will end up with double periods in the final log message which doesn\u0027t seem like a good idea.","commit_id":"50ec89be1c34f43769332a78cce9305256d007ea"}]}