Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the possibility to archive metadata for deleted DIDs #5484

Open
cserf opened this issue Apr 15, 2022 · 2 comments · May be fixed by #6727
Open

Add the possibility to archive metadata for deleted DIDs #5484

cserf opened this issue Apr 15, 2022 · 2 comments · May be fixed by #6727

Comments

@cserf
Copy link
Contributor

cserf commented Apr 15, 2022

Motivation

Currently when a DID is deleted, it is archived in the deleted_did table. The metadata stored in the dids are therefore archived, but not the one stored in json. We should have the possibility to choose to archive metadata once a DID is deleted.

@bari12 bari12 changed the title Metadata : Add the possibility to archive metadata for deleted DIDs Add the possibility to archive metadata for deleted DIDs May 13, 2022
@bari12
Copy link
Member

bari12 commented May 18, 2022

Just to quickly summarise what @cserf @robbarnsley and I discussed today:
To implement this feature the idea is to add a on_delete trigger to the metadata plugin interface. On the deletion of a did, this trigger is then called for each plugin and the plugin itself decides how to best archive the metadata (Or might not do that at all, if not supported by the plugin)

The difficult part is for the column_metadata plugin the data is already moved to the delete_dids table. However, this is mainly done to block the scope, name pair from being re-used in the future and only as a side-effect it archives the column metadata as well. Thus the case needs to be foreseen that a Rucio instance might not want to archive column metadata, but still block a did from being re-used in the future, so this needs to be foreseen in the code.

@cserf will work on a design suggestion on the interface in the next weeks.

@robbarnsley
Copy link
Contributor

Looks like the PR hit a roadblock - are you still working on this @cserf ?

Happy to help if I can. Would be very helpful to have this functionality.

@robbarnsley robbarnsley linked a pull request Apr 10, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment