)]}'
{"nova/conductor/manager.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5825087537c66b81c51abf36fa3be04f6e1dda67","unresolved":false,"context_lines":[{"line_number":906,"context_line":"                    context, instance.uuid)"},{"line_number":907,"context_line":"            except exception.BuildRequestNotFound:"},{"line_number":908,"context_line":"                # the build request is gone so we\u0027re done for this instance"},{"line_number":909,"context_line":"                LOG.debug(\u0027While scheduling instance, the build request \u0027"},{"line_number":910,"context_line":"                          \u0027was already deleted.\u0027, instance\u003dinstance)"},{"line_number":911,"context_line":"                continue"},{"line_number":912,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_bc3ba237","line":909,"updated":"2017-02-02 13:48:36.000000000","message":"I see this debug log message in a run on this patch (3 times in this log):\n\nhttp://logs.openstack.org/75/427775/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/c906c15/logs/screen-n-cond.txt.gz#_2017-02-01_16_14_03_273\n\nSo it\u0027s obviously catching the issue.\n\nhttps://review.openstack.org/#/c/427782/ merged last night but we\u0027re still seeing failures after it merged in this patch:\n\nhttps://review.openstack.org/#/c/427267/\n\nSo I think we should get this patch merged for Ocata given today is rc1.","commit_id":"8ba92778fe14b47ad4ff5b53022e0550a93f37d3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"cd6d8a39ee4b1572d68f97b3b550b1fa26f39852","unresolved":false,"context_lines":[{"line_number":908,"context_line":"                # the build request is gone so we\u0027re done for this instance"},{"line_number":909,"context_line":"                LOG.debug(\u0027While scheduling instance, the build request \u0027"},{"line_number":910,"context_line":"                          \u0027was already deleted.\u0027, instance\u003dinstance)"},{"line_number":911,"context_line":"                continue"},{"line_number":912,"context_line":"            else:"},{"line_number":913,"context_line":"                with obj_target_cell(instance, cell):"},{"line_number":914,"context_line":"                    instance.create()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_d63a3198","line":911,"updated":"2017-02-01 15:59:58.000000000","message":"Do we need to change the notification we emit when this happens or is this taken care of further down the stack?","commit_id":"8ba92778fe14b47ad4ff5b53022e0550a93f37d3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f800ce3b08b705ce0e9548f720bd259a00432366","unresolved":false,"context_lines":[{"line_number":908,"context_line":"                # the build request is gone so we\u0027re done for this instance"},{"line_number":909,"context_line":"                LOG.debug(\u0027While scheduling instance, the build request \u0027"},{"line_number":910,"context_line":"                          \u0027was already deleted.\u0027, instance\u003dinstance)"},{"line_number":911,"context_line":"                continue"},{"line_number":912,"context_line":"            else:"},{"line_number":913,"context_line":"                with obj_target_cell(instance, cell):"},{"line_number":914,"context_line":"                    instance.create()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_2787c1d8","line":911,"in_reply_to":"3a461143_0725655b","updated":"2017-02-01 17:18:50.000000000","message":"Darn it - I read that wrong. My bad","commit_id":"8ba92778fe14b47ad4ff5b53022e0550a93f37d3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"aecc70a8e7534ca224a079553380028754e305fe","unresolved":false,"context_lines":[{"line_number":908,"context_line":"                # the build request is gone so we\u0027re done for this instance"},{"line_number":909,"context_line":"                LOG.debug(\u0027While scheduling instance, the build request \u0027"},{"line_number":910,"context_line":"                          \u0027was already deleted.\u0027, instance\u003dinstance)"},{"line_number":911,"context_line":"                continue"},{"line_number":912,"context_line":"            else:"},{"line_number":913,"context_line":"                with obj_target_cell(instance, cell):"},{"line_number":914,"context_line":"                    instance.create()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_0725655b","line":911,"in_reply_to":"3a461143_d63a3198","updated":"2017-02-01 17:07:50.000000000","message":"The notification doesn\u0027t happen since we\u0027re continuing, right?","commit_id":"8ba92778fe14b47ad4ff5b53022e0550a93f37d3"}]}
