)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"f4b90d0276a78ff6e61034f0c5dd5a03bf5a7a01","unresolved":false,"context_lines":[{"line_number":17,"context_line":"signals."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Unit test was added to verify parent stop method is called in"},{"line_number":20,"context_line":"a shutdown condition."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I21a45862f491530523037632fd70c60cc0153c0a"},{"line_number":23,"context_line":"Implements: blueprint graceful-shutdownw:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAS3%2F%2F1IM%3D","line":20,"updated":"2013-09-18 15:30:22.000000000","message":"Add pls here commit\u0027s hash from oslo-incubator that changes this file:\n\nnova/openstack/common/service.py","commit_id":"eab47f4e76d45af18af69af062b00c58082e5be7"},{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"04d85205d1ee28044f5f8df3647e4afc8201f6f0","unresolved":false,"context_lines":[{"line_number":20,"context_line":"a shutdown condition."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I21a45862f491530523037632fd70c60cc0153c0a"},{"line_number":23,"context_line":"Implements: blueprint graceful-shutdownw:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAS3%2F%2F1S0%3D","line":23,"updated":"2013-09-18 14:57:55.000000000","message":"Typo here (extra \u0027w:\u0027)","commit_id":"eab47f4e76d45af18af69af062b00c58082e5be7"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"4ae493283975696bd52472c99741a57042ef527f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"waits on all child threads (Proxy callbacks) to complete."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The oslo-incubator commit hash that was used to update"},{"line_number":16,"context_line":"nova was acf28bc0fc5a2357bb824eab3995fcbb02c05ac7."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This allows long running commands to complete prior to killing"},{"line_number":19,"context_line":"the process.  This change only works for SIGTERM and SIGINT"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2FoN8%3D","line":16,"updated":"2013-09-23 18:29:40.000000000","message":"seems like you pasted a wrong commit id here.","commit_id":"7937ef324a8b0ed3c3c5d97f0bb2fe984b24ab31"},{"author":{"_account_id":8430,"name":"Christopher Lefelhocz","email":"christopher.lefelhoc@rackspace.com","username":"leif"},"change_message_id":"b6a1d6dfee31003da428fba7fe9ca4711af1d6cc","unresolved":false,"context_lines":[{"line_number":13,"context_line":"waits on all child threads (Proxy callbacks) to complete."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The oslo-incubator commit hash that was used to update"},{"line_number":16,"context_line":"nova was acf28bc0fc5a2357bb824eab3995fcbb02c05ac7."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This allows long running commands to complete prior to killing"},{"line_number":19,"context_line":"the process.  This change only works for SIGTERM and SIGINT"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2Fn7E%3D","line":16,"in_reply_to":"AAAAS3%2F%2FoN8%3D","updated":"2013-09-23 19:45:22.000000000","message":"Technically this was head when I updated the file.  However, I\u0027ll adjust to my actual commit hash.  There were no changes to the file in the interim.","commit_id":"7937ef324a8b0ed3c3c5d97f0bb2fe984b24ab31"}],"nova/openstack/common/service.py":[{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"04d85205d1ee28044f5f8df3647e4afc8201f6f0","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        if pid \u003d\u003d 0:"},{"line_number":276,"context_line":"            # NOTE(johannes): All exceptions are caught to ensure this"},{"line_number":277,"context_line":"            # doesn\u0027t fallback into the loop spawning children. It would"},{"line_number":278,"context_line":"            # be bad for a child to spawn more children."},{"line_number":279,"context_line":"            launcher \u003d self._child_process(wrap.service)"},{"line_number":280,"context_line":"            while True:"},{"line_number":281,"context_line":"                self._child_process_handle_signal()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F1Sg%3D","line":278,"updated":"2013-09-18 14:57:55.000000000","message":"I\u0027m guessing this is from upstream, but this comment doesn\u0027t match the code anymore","commit_id":"eab47f4e76d45af18af69af062b00c58082e5be7"},{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"21b2fc58393d77d496fdc9442cf1d3391aece138","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        if pid \u003d\u003d 0:"},{"line_number":276,"context_line":"            # NOTE(johannes): All exceptions are caught to ensure this"},{"line_number":277,"context_line":"            # doesn\u0027t fallback into the loop spawning children. It would"},{"line_number":278,"context_line":"            # be bad for a child to spawn more children."},{"line_number":279,"context_line":"            launcher \u003d self._child_process(wrap.service)"},{"line_number":280,"context_line":"            while True:"},{"line_number":281,"context_line":"                self._child_process_handle_signal()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F0gQ%3D","line":278,"in_reply_to":"AAAAS3%2F%2F0io%3D","updated":"2013-09-18 17:38:32.000000000","message":"The comment serves a purpose. Please make sure it gets moved to where the code it\u0027s referring to got moved to. (Up in _child_wait_for_exit_or_signal)","commit_id":"eab47f4e76d45af18af69af062b00c58082e5be7"},{"author":{"_account_id":8430,"name":"Christopher Lefelhocz","email":"christopher.lefelhoc@rackspace.com","username":"leif"},"change_message_id":"1a7f9fbdf6a3382a1fa0fd3ded1b1cd3fb9640fc","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        if pid \u003d\u003d 0:"},{"line_number":276,"context_line":"            # NOTE(johannes): All exceptions are caught to ensure this"},{"line_number":277,"context_line":"            # doesn\u0027t fallback into the loop spawning children. It would"},{"line_number":278,"context_line":"            # be bad for a child to spawn more children."},{"line_number":279,"context_line":"            launcher \u003d self._child_process(wrap.service)"},{"line_number":280,"context_line":"            while True:"},{"line_number":281,"context_line":"                self._child_process_handle_signal()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2F0io%3D","line":278,"in_reply_to":"AAAAS3%2F%2F1Sg%3D","updated":"2013-09-18 17:31:41.000000000","message":"I\u0027ll look into removing comment upstream.","commit_id":"eab47f4e76d45af18af69af062b00c58082e5be7"}]}
