)]}'
{"glance/domain/__init__.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"70d6bec6b5534be5474566303e48a6bf5f149002","unresolved":true,"context_lines":[{"line_number":115,"context_line":"        # never committed, hence still visible and \"undeletable\") can be"},{"line_number":116,"context_line":"        # re-deleted through the normal API path, which re-runs image_destroy"},{"line_number":117,"context_line":"        # and finally sets deleted\u003d1. See LP bug #1452788."},{"line_number":118,"context_line":"        \u0027deleted\u0027: (\u0027deleted\u0027,),"},{"line_number":119,"context_line":"        \u0027deactivated\u0027: (\u0027active\u0027, \u0027deleted\u0027),"},{"line_number":120,"context_line":"    }"},{"line_number":121,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"80a863ef_e09226bb","line":118,"range":{"start_line":118,"start_character":29,"end_line":118,"end_character":30},"updated":"2026-06-22 16:17:46.000000000","message":"This is nice, but it feels a bit like a workaround and we\u0027re going to purposefully use a \"wrong\" graph here :/ \n\nHave you thought about combining both operations in a single atomic write on the DB? Even though that would probably mean we could not keep one operation in the domain layer and the other one in the DB layer. \n\nI\u0027m not sure what the right approach is here, I\u0027m just trying to brainstorm this.","commit_id":"6cebb331a0840fc21505bf0907b9f1d137f2b9f5"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"7b4e71f0e6f03d32901a253c1a72b34a34df5585","unresolved":true,"context_lines":[{"line_number":115,"context_line":"        # never committed, hence still visible and \"undeletable\") can be"},{"line_number":116,"context_line":"        # re-deleted through the normal API path, which re-runs image_destroy"},{"line_number":117,"context_line":"        # and finally sets deleted\u003d1. See LP bug #1452788."},{"line_number":118,"context_line":"        \u0027deleted\u0027: (\u0027deleted\u0027,),"},{"line_number":119,"context_line":"        \u0027deactivated\u0027: (\u0027active\u0027, \u0027deleted\u0027),"},{"line_number":120,"context_line":"    }"},{"line_number":121,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"8d29f49a_e68ac539","line":118,"range":{"start_line":118,"start_character":29,"end_line":118,"end_character":30},"in_reply_to":"42fa08d0_408fef4f","updated":"2026-06-23 15:58:16.000000000","message":"OK. Maybe we could discuss this during the upstream weekly meeting on Thursday at 1400 UTC? It will be on #openstack-glance on OFTC.","commit_id":"6cebb331a0840fc21505bf0907b9f1d137f2b9f5"},{"author":{"_account_id":34378,"name":"Julien LE JEUNE","email":"julien.le-jeune@mailops.fr","username":"jlejeune"},"change_message_id":"58bd15f397513f88d7b2ab30abe55ae8e44a60f7","unresolved":true,"context_lines":[{"line_number":115,"context_line":"        # never committed, hence still visible and \"undeletable\") can be"},{"line_number":116,"context_line":"        # re-deleted through the normal API path, which re-runs image_destroy"},{"line_number":117,"context_line":"        # and finally sets deleted\u003d1. See LP bug #1452788."},{"line_number":118,"context_line":"        \u0027deleted\u0027: (\u0027deleted\u0027,),"},{"line_number":119,"context_line":"        \u0027deactivated\u0027: (\u0027active\u0027, \u0027deleted\u0027),"},{"line_number":120,"context_line":"    }"},{"line_number":121,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"42fa08d0_408fef4f","line":118,"range":{"start_line":118,"start_character":29,"end_line":118,"end_character":30},"in_reply_to":"80a863ef_e09226bb","updated":"2026-06-23 06:59:58.000000000","message":"I agree that\u0027s more a workaround than a fix. I suggested this patch to get the discussion started 😉\nBecause this bug has been around since the Juno release now and it\u0027s complicated to combine both operations in a single atomic write.","commit_id":"6cebb331a0840fc21505bf0907b9f1d137f2b9f5"}]}