)]}'
{"nova/cmd/manage.py":[{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"a47bf2622a908dfbfa4ad12b219fe6ad1b2b8c90","unresolved":false,"context_lines":[{"line_number":1491,"context_line":"                  cell_uuid)"},{"line_number":1492,"context_line":"            return 2"},{"line_number":1493,"context_line":""},{"line_number":1494,"context_line":"        # Check to see if there are any InstanceMappings for this cell."},{"line_number":1495,"context_line":"        instance_mappings \u003d objects.InstanceMappingList.get_by_cell_id("},{"line_number":1496,"context_line":"            ctxt, cell_mapping.id)"},{"line_number":1497,"context_line":"        if instance_mappings:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a3c09a3_3eb82ec8","line":1494,"updated":"2017-01-17 00:05:49.000000000","message":"What does it mean to have a cell with no hosts but instances? Does that imply these two should be reversed?","commit_id":"3e04a32df4d06f79a11423c9c0a7780d065187b1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2c2088f5e19e26d7cb1a65a5c2cd510db67262c4","unresolved":false,"context_lines":[{"line_number":1491,"context_line":"                  cell_uuid)"},{"line_number":1492,"context_line":"            return 2"},{"line_number":1493,"context_line":""},{"line_number":1494,"context_line":"        # Check to see if there are any InstanceMappings for this cell."},{"line_number":1495,"context_line":"        instance_mappings \u003d objects.InstanceMappingList.get_by_cell_id("},{"line_number":1496,"context_line":"            ctxt, cell_mapping.id)"},{"line_number":1497,"context_line":"        if instance_mappings:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a3c09a3_b95a7828","line":1494,"in_reply_to":"7a3c09a3_3eb82ec8","updated":"2017-01-17 00:22:14.000000000","message":"Just like you can delete compute_nodes records and instances still point at them today, you can technically delete host_mappings to a cell that has instances pointing to that cell. It\u0027s weird I know, and this command is just checking for what makes an \u0027empty\u0027 cell for the delete. We don\u0027t currently have any ways to delete an instance mapping or host mapping, or move those between cells, but at some point in the future I\u0027m sure we will.\n\nThere is also no FKey constraint between the host and instance mappings in the data model so I\u0027m checking them separately.\n\nThere are Fkeys between the instance/host mappings and their cell mapping, so simply trying to delete the cell mapping if it has referential constraints should fail, but I figured doing this in a more straight-forward way was a bit nicer for the error message...","commit_id":"3e04a32df4d06f79a11423c9c0a7780d065187b1"}]}