)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c05637f88e2012d589a22047bedbde754cfbc456","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Aleksey Ruban \u003caleksey.ruban@nexenta.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-23 08:08:28 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Changes for Newton release"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replaced nas_ip option with nas_host in NFS driver for NexentaStor 5"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bab6814e_4cc22986","line":7,"updated":"2016-05-23 15:07:16.000000000","message":"This is not a good commit summary message. It was fixed in the last update, but added back in now. Please change.","commit_id":"723b98de8be3654a61f38dda3641323a9ad62b1d"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c05637f88e2012d589a22047bedbde754cfbc456","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replaced nas_ip option with nas_host in NFS driver for NexentaStor 5"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Added NexentaEdge NBD driver."},{"line_number":12,"context_line":"The driver should be on the same host where NexentaEdge management is"},{"line_number":13,"context_line":"located."},{"line_number":14,"context_line":"Typical configuration in /etc/cinder/cinder.cfg:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bab6814e_0c430114","line":11,"updated":"2016-05-23 15:07:16.000000000","message":"New driver addition should be in its own patch. It also needs a link to the blueprint for the new driver.","commit_id":"723b98de8be3654a61f38dda3641323a9ad62b1d"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"415e03e0afbf036b9a88903ff4afddaf4fd32851","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Aleksey Ruban \u003caleksey.ruban@nexenta.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-24 09:20:21 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NexentaStor 5 NFS backend driver."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replaced nas_ip option with nas_host in NFS driver for NexentaStor 5"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"bab6814e_87baeabe","line":7,"updated":"2016-05-24 15:24:39.000000000","message":"This does not describe what this commit is.  (It sounds like it\u0027s adding a new driver.)","commit_id":"dbfb1f726c38f570487866b42d6f7dadcccec50e"}],"cinder/volume/drivers/nexenta/nexentaedge/jsonrpc.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"415e03e0afbf036b9a88903ff4afddaf4fd32851","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        elif self.method \u003d\u003d \u0027delete\u0027:"},{"line_number":94,"context_line":"            req \u003d requests.delete(self.url, data\u003ddata, headers\u003dheaders)"},{"line_number":95,"context_line":"        else:"},{"line_number":96,"context_line":"            raise Exception(\u0027Unsupported method: %s\u0027 % self.method)"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        rsp \u003d req.json()"},{"line_number":99,"context_line":"        req.close()"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_07a0ba8f","line":96,"updated":"2016-05-24 15:24:39.000000000","message":"Raise a VolumeDriverException instead of Exception.","commit_id":"dbfb1f726c38f570487866b42d6f7dadcccec50e"}],"cinder/volume/drivers/nexenta/ns5/nfs.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"415e03e0afbf036b9a88903ff4afddaf4fd32851","unresolved":false,"context_lines":[{"line_number":332,"context_line":"                            \u0027filesystem\u0027: volume[\u0027name\u0027]})"},{"line_number":333,"context_line":"            raise"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"        if ((\u0027size\u0027 in volume) and ("},{"line_number":336,"context_line":"                volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027])):"},{"line_number":337,"context_line":"            self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":338,"context_line":"        return {\u0027provider_location\u0027: volume[\u0027provider_location\u0027]}"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_a7a0cea6","line":335,"updated":"2016-05-24 15:24:39.000000000","message":"You should assume that \u0027size\u0027 is present in a volume ref rather than checking for it.  (It is a required field.)","commit_id":"dbfb1f726c38f570487866b42d6f7dadcccec50e"}]}