)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"3e938375db0d53f74e2bb891985fa7a99c7c97d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"8185d091_cd34c271","updated":"2026-04-08 12:39:56.000000000","message":"Well commented, well structured patch. LGTM","commit_id":"6dbda8f9b91430cd900dc543edddd7148cb94374"}],"tempest/api/compute/admin/test_live_migration.py":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"3e938375db0d53f74e2bb891985fa7a99c7c97d9","unresolved":true,"context_lines":[{"line_number":686,"context_line":"                   self.os_admin.servers_client.list_instance_actions("},{"line_number":687,"context_line":"                       server_id)[\u0027instanceActions\u0027]"},{"line_number":688,"context_line":"                   if action[\u0027action\u0027] \u003d\u003d \u0027live-migration\u0027]"},{"line_number":689,"context_line":"        latest_action \u003d max(actions, key\u003dlambda a: a[\u0027updated_at\u0027])"},{"line_number":690,"context_line":""},{"line_number":691,"context_line":"        # Verify the migration failure was for secret permissions."},{"line_number":692,"context_line":"        action \u003d self.os_admin.servers_client.show_instance_action("}],"source_content_type":"text/x-python","patch_set":11,"id":"5485fac3_9c4a1aa4","line":689,"updated":"2026-04-08 12:39:56.000000000","message":"Minor nit since \u0027actions\u0027 is always expected to be populated, but it would be good practice to raise an error if actions is empty. Something like:\n```\nif not actions:\n    self.fail(\"No \u0027live-migration\u0027 instance action found\")\n```\nBut I won\u0027t add a -1 for this","commit_id":"6dbda8f9b91430cd900dc543edddd7148cb94374"}]}