)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"98f0f90c928afea32a581c5af3cfa0480702dbba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eef0efea_543c2b17","updated":"2025-04-28 21:52:24.000000000","message":"recheck massive ceph failure and obvious functional unrelated fail","commit_id":"e451d8771dedb33a8d0210df5fedfaa6534e64f5"}],"nova/cmd/manage.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8273fd39ad7d90ce48807b8ec30f5b9184865d39","unresolved":true,"context_lines":[{"line_number":215,"context_line":"        compute_node_obj.migrate_empty_ratio,"},{"line_number":216,"context_line":"        # Added in Stein"},{"line_number":217,"context_line":"        virtual_interface_obj.fill_virtual_interface_list,"},{"line_number":218,"context_line":"        # Added in Stein"},{"line_number":219,"context_line":"        instance_mapping_obj.populate_user_id,"},{"line_number":220,"context_line":"        # Added in Victoria"},{"line_number":221,"context_line":"        pci_device_obj.PciDevice.populate_dev_uuids,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f1ed5a1_120a3b28","side":"PARENT","line":218,"updated":"2025-04-28 14:48:35.000000000","message":"Note to reviewers, we chatted about this approach a little on irc\n\nWhile we can remove the other old migration and probably should in a separate patch\n\ndan has intentionally kept this small so we could backport to Carical.\n\nwith this change, you need ot have archive the deleted rows to remove the fake marker instance before running the online  data migration on carical (technically bobcat but that is being EOL\u0027d as we speak) or later.\n\n\nso the proposal is a follow, \n1 have a minimal patch to remove the problematic migration for moving to stein.\n2 backport that to the oldest supprot slurp release aka 2024.1 (carical)\n3 have a seperate followup commit to clean up other old online migrations when we ahve time on master only.\n\ni.e. everything before \n```\n# Added in 2023.2\n        instance_obj.populate_instance_compute_id,\n```\n\n\n@dms@danplanet.com, we might consider a fixes or upgrade release note just to call out for those doing a slrup or general upgrade that this issue has now been resolved. We should then plan to get this released in stable once backported, but I\u0027ll leave that to our discussion if you think that is needed or not.","commit_id":"3e7017eb2952d5258d96e27b6c16aae1ed472b21"}]}