)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"change_message_id":"f90341b407f04f758250119f882947663fb0293e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ian Denhardt \u003cian@zenhack.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-03-25 21:47:47 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix incorrect invocation of _add_to_threadpool."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"_add_to_threadpool expects to be passed the function and which is to be"},{"line_number":10,"context_line":"run in an alternate thread, and the function\u0027s arguments. Instead, the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6ced46_929df4be","line":7,"updated":"2015-03-26 12:13:45.000000000","message":"I am not sure if the convention still applies, but if it is still valid, you cannot have a period at the end of the commit message. (https://wiki.openstack.org/wiki/GitCommitMessages)","commit_id":"df0cb7fec87013d05564f7de0880f558ac538545"}],"cinder/volume/manager.py":[{"author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"change_message_id":"f90341b407f04f758250119f882947663fb0293e","unresolved":false,"context_lines":[{"line_number":362,"context_line":"                    # threadpool to prevent the main volume service thread"},{"line_number":363,"context_line":"                    # from being blocked."},{"line_number":364,"context_line":"                    self._add_to_threadpool(self.delete_volume, ctxt,"},{"line_number":365,"context_line":"                                                               volume[\u0027id\u0027])"},{"line_number":366,"context_line":"                else:"},{"line_number":367,"context_line":"                    # By default, delete volumes sequentially"},{"line_number":368,"context_line":"                    self.delete_volume(ctxt, volume[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_d21e8c5e","line":365,"updated":"2015-03-26 12:13:45.000000000","message":"You will need to align volume[\u0027id\u0027] with the first argument (self.delete_volume). Jenkins\u0027 pep8 will not let it pass with this indent.","commit_id":"df0cb7fec87013d05564f7de0880f558ac538545"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"a0410d63661e52230fa58198e85f359e2d9d1f63","unresolved":false,"context_lines":[{"line_number":362,"context_line":"                    # threadpool to prevent the main volume service thread"},{"line_number":363,"context_line":"                    # from being blocked."},{"line_number":364,"context_line":"                    self._add_to_threadpool(self.delete_volume, ctxt,"},{"line_number":365,"context_line":"                                                                volume[\u0027id\u0027])"},{"line_number":366,"context_line":"                else:"},{"line_number":367,"context_line":"                    # By default, delete volumes sequentially"},{"line_number":368,"context_line":"                    self.delete_volume(ctxt, volume[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_bd969892","line":365,"updated":"2015-03-26 17:07:37.000000000","message":"I think you still have the pep8 problem Kallebe pointed out. You need:\n                    self._add_to_threadpool(self.delete_volume, ctxt,\n                                            volume[\u0027id\u0027])","commit_id":"f4a46a82652554340dec289221c51177de62a8ce"}]}
