Change abandoned by Matt Riedemann (<email address hidden>) on branch: master
Review: https://review.opendev.org/575556
Reason: I'm going to drop this in favor of https://review.opendev.org/#/c/666438/ and https://review.opendev.org/#/c/655908/. There could still be unaccounted for cases where the build request is gone, the instance is in a cell and the instance mapping is not pointing at a cell (maybe the instance mapping update was dropped by the DB or something - these aren't atomic operations). We can try to shore up more of that flow, e.g. https://review.opendev.org/#/c/586713/ but I'm going to drop this in the meantime since it was really just for debug purposes.
Change abandoned by Matt Riedemann (<email address hidden>) on branch: master /review. opendev. org/575556 /review. opendev. org/#/c/ 666438/ and https:/ /review. opendev. org/#/c/ 655908/. There could still be unaccounted for cases where the build request is gone, the instance is in a cell and the instance mapping is not pointing at a cell (maybe the instance mapping update was dropped by the DB or something - these aren't atomic operations). We can try to shore up more of that flow, e.g. https:/ /review. opendev. org/#/c/ 586713/ but I'm going to drop this in the meantime since it was really just for debug purposes.
Review: https:/
Reason: I'm going to drop this in favor of https:/