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

Fix the Infinite AIPs feature #119

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jrwdunham
Copy link
Contributor

@jrwdunham jrwdunham commented Jun 6, 2018

Fixes #118
Fixes #114
Fixes #117

  • Adds exponential backoff to the polling which detects the availability of the "approve transfer" select.
  • Makes normalization configurable in infinite-aips.feature.

When running ``behave``, you can now pass a tag like
``--tags=decision.id.not-norm`` or one like
``--tags=decision.id.norm-pres`` to control whether normalization is
performed.
@jrwdunham
Copy link
Contributor Author

@jhsimpson I have tested this by running the appropriate behave command on our bcadmin server against the http://158.69.86.69:81/ test machine. Note the new --tags=decision.id.not-norm in the behave command below (which, aside from the false password, should work on our test infra):

$ behave \
      --tags=infinite-aips \
      --tags=decision.id.not-norm \
      --no-skipped \
      --no-logcapture \
      -D runtime_supplied_transfer_path='~/test' \
      -D driver_name=Firefox \
      -D am_url=http://158.69.86.69:81/ \
      -D am_password=somepassword \
      -D am_version=1.7 \
      -D home=archivematica

Previously we were searching in Archival Storage but we want this
feature to be able to work in an "Indexless" (Elasticsearch-less)
deploy.
- Indicate in comments of Infinite AIPs feature file how to use the
  `-D max_check_aip_stored_attempts=86400` to get behave to wait up to a
  day for an AIP to be created.
- Fix environment.py so that user-supplied attempt counts are coerced to
  ints and user-supplied wait values are coerced to floats.
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.

1 participant