Add reproducer for bug #2099702: flavor unset --project fails for deleted projects

When a project has been deleted from Keystone, `openstack flavor unset
--project <deleted-project-id>` fails because find_project_id_sdk()
validates the project against Keystone with validate_actor_existence=True
(the default) and raises CommandError when the project is not found.

This means Nova's removeTenantAccess API is never called, effectively
bypassing the server-side fix for bug #1980845 which already handles
deleted projects gracefully.

The fix should pass validate_actor_existence=False when calling
find_project_id_sdk() for the removeTenantAccess case, similar to
how RemoveRole handles this in identity/v3/role.py.

Related-Bug: #2099702

Assisted-By: Claude-Code opus 4.6
Change-Id: I438244dd226173b61e5f7c3105fe3006829a0e65
Signed-off-by: René Ribaud <rene.ribaud@gmail.com>

Author: uggla

openstackclient/tests/unit/compute/v2/test_flavor.py

@@ -1222,6 +1222,37 @@ def test_flavor_unset_with_unexist_flavor(self):
             exceptions.CommandError, self.cmd.take_action, parsed_args
         )
 
+    def test_flavor_unset_project_deleted_project(self):
+        # Simulate a project that has been deleted from Keystone
+        self.identity_sdk_client.find_project.side_effect = [
+            sdk_exceptions.ResourceNotFound()
+        ]
+
+        deleted_project_id = 'deleted-project-uuid'
+        arglist = [
+            '--project',
+            deleted_project_id,
+            self.flavor.id,
+        ]
+        verifylist = [
+            ('project', deleted_project_id),
+            ('flavor', self.flavor.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        # TODO(bug #2099702): This should succeed. When removing a project
+        # from a flavor's access list, the project may no longer exist in
+        # Keystone (deleted tenant). The command should pass the raw project
+        # ID through to Nova's removeTenantAccess API, which already handles
+        # this case since the fix for bug #1980845. Instead, the command
+        # fails because find_project_id_sdk() validates the project against
+        # Keystone with validate_actor_existence=True (the default) and
+        # raises CommandError when it doesn't exist.
+        self.assertRaises(
+            exceptions.CommandError, self.cmd.take_action, parsed_args
+        )
+        self.compute_client.flavor_remove_tenant_access.assert_not_called()
+
     def test_flavor_unset_nothing(self):
         arglist = [
             self.flavor.id,