)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"724ad27d0da5cb1ec7dfbda9eb147d0112fc1b9f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"volume one time. If this fails we should retry until it works or"},{"line_number":12,"context_line":"we run out of scserver objects to try."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I383980a8241ec871313e6e7d7157f7af264fc5b6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a30ddce_b1ab4d74","line":14,"updated":"2017-03-10 13:37:13.000000000","message":"-1: missing LP bug","commit_id":"90634f300a6fa063633105e9d07a7a1360fae255"}],"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"724ad27d0da5cb1ec7dfbda9eb147d0112fc1b9f","unresolved":false,"context_lines":[{"line_number":865,"context_line":"                    # Unmap"},{"line_number":866,"context_line":"                    self.unmap_volume(scvolume, scserver)"},{"line_number":867,"context_line":"                    # Did it work?"},{"line_number":868,"context_line":"                    if not scvolume.get(\u0027active\u0027, False):"},{"line_number":869,"context_line":"                        LOG.debug(\u0027Failed to activate volume %(name)s via  \u0027"},{"line_number":870,"context_line":"                                  \u0027server %(srvr)s)\u0027,"},{"line_number":871,"context_line":"                                  {\u0027name\u0027: scvolume[\u0027name\u0027],"},{"line_number":872,"context_line":"                                   \u0027srvr\u0027: scserver[\u0027name\u0027]})"},{"line_number":873,"context_line":"                    else:"},{"line_number":874,"context_line":"                        return"},{"line_number":875,"context_line":"        # We didn\u0027t map/unmap the volume.  So no initialization done."},{"line_number":876,"context_line":"        # Warn the user before we leave.  Note that this is almost certainly"},{"line_number":877,"context_line":"        # a tempest test failure we are trying to catch here.  A snapshot"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a30ddce_31975d3b","line":874,"range":{"start_line":868,"start_character":0,"end_line":874,"end_character":30},"updated":"2017-03-10 13:37:13.000000000","message":"nit: This would look better the other way around:\n\n if scvolume.get(\u0027active\u0027, False):\n     return\n       \n LOG.debug(\u0027Failed to activate volume %(name)s via  \u0027\n           \u0027server %(srvr)s)\u0027,\n           {\u0027name\u0027: scvolume[\u0027name\u0027],\n            \u0027srvr\u0027: scserver[\u0027name\u0027]})","commit_id":"90634f300a6fa063633105e9d07a7a1360fae255"}]}