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

Prototype: Create TAR space in storage service #574

Closed
wants to merge 4 commits into from

Conversation

ross-spencer
Copy link
Contributor

@ross-spencer ross-spencer commented Mar 8, 2021

Draft PR that may be shaped into something more useful for NHA over the next few weeks...

Currently written

  • Tape archive Space migration.
  • Tape archive outline code.

TODO

  • Tests.
  • Pointer file extensions, e.g. new composition level. Contingent on composition levels below.
  • Fallback code, e.g. if Tape-archive extract fails, e.g. attempt reingest, then attempt again.

Questions

  1. See Santi's below: Naming of the TAR after storing. Currently, AIP is stored, then the tar is created and replaces the original AIP in the storage service.
  2. Is the pointer file extension as simple as adding a composition level?
  3. Should the Space be used for an AIPstore and Replicator? (my vote: Yes but see P1 issue below).

Related issues and priorities (P1 high, P5 low):

Testing checks:

  • Ingest (Uncompressed, 7z, tar.gz).
  • METS download.
  • Fixity.
  • Re-ingest and update.
  • Replications update correctly.

@ross-spencer ross-spencer force-pushed the dev/nha-replication-space branch 2 times, most recently from a15a14a to 500d543 Compare March 8, 2021 16:56
@scollazo
Copy link
Member

scollazo commented Mar 9, 2021

After untaring the AIP, the folder name is tar-<UUID>. Should it be <UUID> to align with the uncompressed AIPs?

@tw4l tw4l force-pushed the dev/nha-replication-space branch 4 times, most recently from ab3254c to 094d1c7 Compare April 22, 2021 18:59
@tw4l tw4l force-pushed the dev/nha-replication-space branch 2 times, most recently from ef83b75 to 7a43c74 Compare April 22, 2021 20:52
@tw4l tw4l force-pushed the dev/nha-replication-space branch from 7a43c74 to ff513dc Compare April 26, 2021 16:03
@tw4l
Copy link
Contributor

tw4l commented Apr 26, 2021

Closed in favour of #593

@tw4l tw4l closed this Apr 26, 2021
@tw4l tw4l deleted the dev/nha-replication-space branch April 26, 2021 18:19
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.

3 participants