Idea #12414
closed
[API] Contents of trashed projects are actually deleted once delete_at is past
Added by Peter Amstutz about 7 years ago.
Updated over 6 years ago.
Release relationship:
Auto
Description
Extend existing sweep process needs to be extended to recursively traverse project hierarchy and delete records from the database where owner_uuid is the project being deleted.
- Target version changed from Arvados Future Sprints to To Be Groomed
- Description updated (diff)
- Target version changed from To Be Groomed to Arvados Future Sprints
- Story points set to 1.0
- Description updated (diff)
- Target version changed from Arvados Future Sprints to To Be Groomed
- Target version changed from To Be Groomed to Arvados Future Sprints
- Target version changed from Arvados Future Sprints to 2018-04-11 Sprint
- Assigned To set to Peter Amstutz
- Assigned To changed from Peter Amstutz to Lucas Di Pentima
- Status changed from New to In Progress
This looks good. I have two small suggestions:
- I think you can write "klass.columns.collect{|c| c.name}" as "klass.columns.collect(&:name)"
- Add a couple of checks to the test "delete projects and their contents in SweepTrashedObjects" that things which shouldn't be deleted, were not deleted.
- Status changed from In Progress to Resolved
Also available in: Atom
PDF