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

Getting checksum for Arkivum from pointer file #216

Merged
merged 2 commits into from
Aug 7, 2017

Conversation

jrwdunham
Copy link
Contributor

@jrwdunham jrwdunham commented Jul 19, 2017

Fixes RM issue 11138 where the SS was attempting to calculate a checksum for a compressed AIP in the staging path of an Arkivum space after the AIP was moved to the Arkivum AIP Storage location, which resulted in a No such file or directory IOError. The new behaviour is to retrieve the size, checksum and checksum algorithm of the compressed AIP from its pointer file instead of looking for it on disk.

@@ -12,6 +12,11 @@
ARKIVUM_DIR = os.path.abspath(os.path.join(FIXTURES_DIR, 'arkivum'))


def get_pkg_uuid_path(package_uuid):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you mind adding a comment here for the reader with an example? E.g.:
b9a07040-4f08-4a9c-b88a-5cc325e17e7e => b9a0/7040/4f08/4a9c/b88a/5cc3/25e1/7e7e

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure

Copy link
Member

@sevein sevein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@qubot qubot force-pushed the dev/issue-11138-arkivum-md5-bug branch from f0531cc to 89028c6 Compare July 20, 2017 23:34
@qubot qubot force-pushed the dev/issue-11138-arkivum-md5-bug branch from 89028c6 to 48623d7 Compare August 4, 2017 19:30
@qubot qubot merged commit 48623d7 into qa/0.x Aug 7, 2017
@nickwilkinson nickwilkinson added this to the 0.11.0 milestone Aug 8, 2017
@qubot qubot deleted the dev/issue-11138-arkivum-md5-bug branch September 11, 2018 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants