)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"c0e577054dabd13756bef242fe01167d5da80fb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c54f9de0_e53f7c75","updated":"2026-11-12 08:33:10.000000000","message":"this patch is so good that it invalidates our trusted image cert reproducer\n```\nTraceback (most recent call last):\n  File \"/home/zuul/src/opendev.org/openstack/nova/nova/tests/functional/libvirt/test_trusted_image_certs.py\", line 60, in test_cold_migrate_loses_trusted_certs\n    self.assertIsNone(server[\u0027trusted_image_certificates\u0027])\n    ~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/nova/.tox/functional-py314-threading/lib/python3.14/site-packages/testtools/testcase.py\", line 531, in assertIsNone\n    self.assertThat(observed, matcher, message)\n    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/nova/.tox/functional-py314-threading/lib/python3.14/site-packages/testtools/testcase.py\", line 704, in assertThat\n    raise mismatch_error\ntesttools.matchers._impl.MismatchError: [\u0027a7244cd7-eeb5-4bb4-93d4-d115519a4fbd\u0027] is not None\n```","commit_id":"04fd6618c0ae2f9b35e7db0fa8629f089d09a867"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"25cbf84eb0ddb0735e1ac0653e60db8a390a8033","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d869214_5da93328","updated":"2026-11-12 15:19:03.000000000","message":"looks good to me but I have a question","commit_id":"a2eb2af9770c1bc9ea61b86e3cd4cce6ce0bf7fa"}],"nova/conductor/tasks/cross_cell_migrate.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"25cbf84eb0ddb0735e1ac0653e60db8a390a8033","unresolved":true,"context_lines":[{"line_number":161,"context_line":"            expected_attrs\u003d["},{"line_number":162,"context_line":"                \u0027metadata\u0027, \u0027system_metadata\u0027, \u0027info_cache\u0027, \u0027security_groups\u0027,"},{"line_number":163,"context_line":"                \u0027flavor\u0027, \u0027numa_topology\u0027, \u0027pci_requests\u0027, \u0027keypairs\u0027, \u0027tags\u0027,"},{"line_number":164,"context_line":"                \u0027vcpu_model\u0027, \u0027device_metadata\u0027, \u0027trusted_certs\u0027, \u0027resources\u0027])"},{"line_number":165,"context_line":"        tags \u003d source_instance.tags"},{"line_number":166,"context_line":"        inst \u003d clone_creatable_object("},{"line_number":167,"context_line":"            self.target_ctx, source_instance, delete_fields\u003d[\u0027tags\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"1751d355_e478420b","line":164,"updated":"2026-11-12 15:19:03.000000000","message":"this means we should add a new attribute here if  the Instance object has a new optional field in a new version but OK. \nMaybe we could check objects.Instance._INSTANCE_EXTRA_FIELDS instead here instead of this list ?","commit_id":"a2eb2af9770c1bc9ea61b86e3cd4cce6ce0bf7fa"}]}
