`Disable` generator forbids superusers

[ppanero]

from invenio_access.permissions import superuser_access , system_identity
from invenio_rdm_records.proxies import current_rdm_records_service

system_identity.provides.add(superuser_access)  # should not be needed
current_rdm_records_service.delete(system_identity, "1131649")

     50 if not self.check_permission(identity, action_name, **kwargs):
---> 51     raise PermissionDeniedError(action_name)

PermissionDeniedError: delete

system_identity.provides

{Need(method='action', value='superuser-access'),
 Need(method='system_role', value='system_process')}

[zzacharo]

Decided to implement a cli command per instance for now, as the proper deletion or records is more complex and it will be part of a future milestone (from the moment this comment is written).

Action

• [ ] Create a CLI command, to delete records.

[ppanero]

Clarification needed @zzacharo a) it will be fixed when that cli is implemented? b) it will not be an issue because we will use a cli? if this is the case, the issue will still exist since I assume the cli will call the service, where permissions are checked (and Disable will block the deletion)

[zzacharo]

By CLI, I was meaning a script utilizing the data models instead of services i.e use it only in our instances until we properly attack the issue of deletions via services. So, the issue with the permission will be fixed when the underlying service layer implements the deletion properly.