)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"661fb048ad524b1a05ead2c33ad0400e7abcac43","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dmitry Tantsur \u003cdtantsur@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-09-29 16:11:17 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix typo in agent validation code"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I3309af3627700e3d051042049b8eacac1c632bd4"},{"line_number":10,"context_line":"Closes-Bug: #1375042"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baa201ad_4618cb84","line":7,"updated":"2014-09-29 14:18:00.000000000","message":"I would argue that it wasn\u0027t necessarily a typo, but anyway. I think just saying \u0027typo\u0027 makes it sound like a spelling mistake in a docstring or comment, whereas in this case, the code breaks.","commit_id":"0b19d02e64bf4ffdbace301ff65d789fa118d476"}],"ironic/tests/drivers/test_agent.py":[{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"fc6ac20a78eec5756e53a2449f038cd5be0b0b2d","unresolved":false,"context_lines":[{"line_number":78,"context_line":"        self.node.driver_info \u003d {}"},{"line_number":79,"context_line":"        self.node.save()"},{"line_number":80,"context_line":"        with task_manager.acquire("},{"line_number":81,"context_line":"                self.context, self.node[\u0027uuid\u0027], shared\u003dFalse) as task:"},{"line_number":82,"context_line":"            self.assertRaises(exception.InvalidParameterValue,"},{"line_number":83,"context_line":"                             self.driver.validate, task)"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_023eacb8","line":81,"updated":"2014-09-29 15:06:23.000000000","message":"A small idea for next cycle\u0027s clean up: remove \"shared\u003dFalse\" from all the code, since it\u0027s the default, and also fake task_manager so it doesn\u0027t needlessly require db interactions in unit tests (if that isn\u0027t done already).","commit_id":"0b19d02e64bf4ffdbace301ff65d789fa118d476"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"661fb048ad524b1a05ead2c33ad0400e7abcac43","unresolved":false,"context_lines":[{"line_number":78,"context_line":"        self.node.driver_info \u003d {}"},{"line_number":79,"context_line":"        self.node.save()"},{"line_number":80,"context_line":"        with task_manager.acquire("},{"line_number":81,"context_line":"                self.context, self.node[\u0027uuid\u0027], shared\u003dFalse) as task:"},{"line_number":82,"context_line":"            self.assertRaises(exception.InvalidParameterValue,"},{"line_number":83,"context_line":"                             self.driver.validate, task)"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_011dcd66","line":81,"updated":"2014-09-29 14:18:00.000000000","message":"nit: this is only a test, but for validate, shared can be True.\n\nHmm, I see that it is False in the test above too.","commit_id":"0b19d02e64bf4ffdbace301ff65d789fa118d476"}]}