search

samvera-deprecated / sufia

[DEPRECATED] Sufia: a fully featured, flexible Samvera repository front-end.
http://sufia.io/
Other
111 stars 78 forks source link

Sufia PCDM Migration verification #2129

Open carolyncole opened 8 years ago

carolyncole commented 8 years ago

I would like to be able to verify that all of my Sufia 6.x object were migrated to my Sufia PCDM repository after migration, so that I can verify that every object has been moved. Part of user story #2127

I would like to create a executable I can run that uses a database table created by sufia_survey to verify the existence and type of every object in the survey (see #2128).

This will be done when I can run sufia_verify.exe in my Sufia PCDM application and it updates the survey database table to include a status for every item in the survey.

My executable should take the following options: --limit= [all , failures] # limit running the verification to either all items, or just the failed items

hackartisan commented 7 years ago

I don't understand the limit option. How do you limit to just failures if you haven't run the entire validation? Or are you saying you want a report to be generated?

carolyncole commented 7 years ago

@hackmastera So in theory there is a list of items that have failed after the survey has run once. When you run the survey do you want to run a survey of all SurveyItems or only the ones that failed the last time you ran a Survey

hackartisan commented 7 years ago

@cam156 This implies a way of creating a report of some kind. Maybe we need a new ticket for that?

carolyncole commented 7 years ago

@hackmastera the database records have the status already. It would be a matter of changing the statement from all to a where clause here: https://github.com/projecthydra/sufia/blob/7.2-migration/lib/sufia/migration/validation/service.rb#L19

It may not be valuable...

hackartisan commented 7 years ago

Oh yes that would be easier than passing a list. Still I think a report might be useful; what do you think @cam156 ? Maybe just a printed summary of the counts / IDs for missing or wrong_class statuses?

hackartisan commented 7 years ago

This is done, except for the 'limit' option. I'm unassigning myself; if someone wants to do that part I'm sure it won't take very long.

mjgiarlo commented 7 years ago

@cam156 @hackmastera Will the remaining work for this ticket go into the 7.3.0 release? If so, any projections when this may be available? I'd like to schedule an initial 7.3.0 release soon after we get back from holiday break.

carolyncole commented 7 years ago

@mjgiarlo all of this work is in the 7.2 migration branch. As of yet there is no plan on how to put that work back into Sufia develop. Unfortunately PSU does not have the cycles in the near term to work on that.

mjgiarlo commented 7 years ago

@cam156 I suspect merging the 7.2 migration branch with the 7.1 stable branch into a new 7.x-stable branch, and cutting the 7.2.1 release from that branch, is the way to go here. I know y'all are focused on SS 3.0 now -- not a problem, and thanks for all the work you're doing!