)]}'
{"tempest/scenario/manager.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aaeac08770dbf0fadd6be989beafa284c4c5dcc5","unresolved":false,"context_lines":[{"line_number":116,"context_line":"            # If the resource is already missing, mission accomplished."},{"line_number":117,"context_line":"            # add status code as workaround for bug 1247568"},{"line_number":118,"context_line":"            if (e.__class__.__name__ in (\u0027NotFound\u0027, \u0027HTTPNotFound\u0027) or"},{"line_number":119,"context_line":"                    (hasattr(e, \u0027status_code\u0027) and e.status_code \u003d\u003d 404)):"},{"line_number":120,"context_line":"                return"},{"line_number":121,"context_line":"            raise"},{"line_number":122,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_10aae684","line":119,"updated":"2014-05-16 06:52:21.000000000","message":"Can line119 cover HTTPNotFound cases?","commit_id":"0e030b5d684adb260f4256475ee2983569544e04"},{"author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"change_message_id":"70d2bf50f43af1ce3e73bc56bae8ad45a804d50a","unresolved":false,"context_lines":[{"line_number":116,"context_line":"            # If the resource is already missing, mission accomplished."},{"line_number":117,"context_line":"            # add status code as workaround for bug 1247568"},{"line_number":118,"context_line":"            if (e.__class__.__name__ in (\u0027NotFound\u0027, \u0027HTTPNotFound\u0027) or"},{"line_number":119,"context_line":"                    (hasattr(e, \u0027status_code\u0027) and e.status_code \u003d\u003d 404)):"},{"line_number":120,"context_line":"                return"},{"line_number":121,"context_line":"            raise"},{"line_number":122,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_66b44dd1","line":119,"in_reply_to":"7adec928_10aae684","updated":"2014-05-16 13:40:02.000000000","message":"My testing indicates it doesn\u0027t, the teardown cleanup in https://review.openstack.org/#/c/90143/ fails without this change.  Possibly there are some api changes needed to ensure the status_code ends up 404 but I\u0027ve not had time to investigate yet, just catching HTTPNotFound seemed the easiest way to solve the problem.","commit_id":"0e030b5d684adb260f4256475ee2983569544e04"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f1bef596f54c39ce7dc1b4975d5cad3055e57803","unresolved":false,"context_lines":[{"line_number":116,"context_line":"            # If the resource is already missing, mission accomplished."},{"line_number":117,"context_line":"            # add status code as workaround for bug 1247568"},{"line_number":118,"context_line":"            if (e.__class__.__name__ in (\u0027NotFound\u0027, \u0027HTTPNotFound\u0027) or"},{"line_number":119,"context_line":"                    (hasattr(e, \u0027status_code\u0027) and e.status_code \u003d\u003d 404)):"},{"line_number":120,"context_line":"                return"},{"line_number":121,"context_line":"            raise"},{"line_number":122,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_790eb23c","line":119,"in_reply_to":"7adec928_66b44dd1","updated":"2014-05-16 14:46:06.000000000","message":"Could you add a comment to explain this?","commit_id":"0e030b5d684adb260f4256475ee2983569544e04"}]}