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

Improve ~remote~ _playbook_ instructions. #2191

Merged
merged 8 commits into from
Jan 19, 2023
Merged

Conversation

rosiel
Copy link
Member

@rosiel rosiel commented Dec 5, 2022

Purpose / why

The instructions are outdated! We no longer need karaf or so many fedora uris.

What changes were made?

  • removed deprecated options
  • updated the sample new folder name from example to production to give it more realism and also to match the command at the bottom

Verification

Can you use this to provision a ubuntu 20.04, and get all the expected behaviours?

Interested Parties

@alxp @whikloj

Checklist

Pull-request Reviewer

Pull-request reviewer should ensure the following:

Person Merging

The person merging should ensure the following:

  • Does mkdocs still build successfully? (This is indicated by TravisCI passing. To test locally, and see warnings, see How To Build Documentation.)
  • If pages are renamed or removed, have all internal links to those pages been fixed?
  • If pages are added, have they been linked to or placed in the menu?
  • Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

@rosiel
Copy link
Member Author

rosiel commented Dec 5, 2022

Current issues:

  1. Can't index into Fedora. After playbook running, there are ~32 messages dequeued from the islandora-indexing-fcrepo-content, but once i expose the Fedora interface (see below) i see none of them are in Fedora. The fcrepo/rest interface shows an empty repository with no "children".

The method I used to open Fedora was the "alternatively" fcrepo.xml file approach here: https://wiki.lyrasis.org/display/FEDORA6x/Disable+Authentication+and+Authorization. Once this has been done, i see new "children" in Fedora when i add content in Drupal. This definitely isn't a step i need to take in Vagrant (for indexing OR for access at :8080), but I'm not sure why. By the time I checked (the playbook had finished) the admin user had fedoraAdmin role.

Here's a sample of (one node's worth of) errors in milliner log.

  1. Known issue, affects Vagrant too: We're putting messages to islandora-indexing-fcrepo-file-external but nothing is reading them: the "Number of Consumers" is 0.

  2. FITS isn't ... working? Using both the automatic Contexts and the Action to create FITS derivatives, I get nothing back. The messages are being dequeued from the ActiveMQ. There's an auth error in the alpaca logs. See error log snippet.

@whikloj
Copy link
Member

whikloj commented Dec 5, 2022

I don't see any lines marked with ERROR in the milliner log snippet you referenced. Is there more after? Also it would be good to see what the Fedora log is saying too.

@rosiel rosiel marked this pull request as ready for review December 6, 2022 21:37
@rosiel
Copy link
Member Author

rosiel commented Dec 6, 2022

I tried a fresh server and the milliner problems didn't happen (Also, wow I think I failed to copy the wrong section of those logs!)

Fits is kind of a different thing, so i'm investigating that separately, but it probably doesn't have to do with the remote deployment instructions.

drupal_db_password: changeme
drupal_account_pass: changeme
islandora_db_root_password: changeme
Copy link
Member Author

Choose a reason for hiding this comment

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

don't worry the rest of the passwords are there on the right... just bumped down in the diff.

```yml
fcrepo_allowed_external_content:
- http://example.org/
cantaloupe_HttpResolver_BasicLookupStrategy_url_prefix: http://example.org/
Copy link
Member Author

Choose a reason for hiding this comment

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

This was breaking cantaloupe, hat-tip to @whikloj for helping find the solution.

@ysuarez
Copy link
Contributor

ysuarez commented Dec 14, 2022

Is this ready for merging, do we need a final person besides Rosie to give it a final "blessing" before we merge?

@rosiel rosiel changed the title Improve remote instructions. Improve ~remote~ _playbook_ instructions. Dec 15, 2022
@rosiel
Copy link
Member Author

rosiel commented Dec 15, 2022

Sorry for the last minute changes - we just updated the interface for using the Playbook so I put the new info here.

@rosiel
Copy link
Member Author

rosiel commented Jan 18, 2023

I've deleted the sneaky files that I added by mistake (thanks whoever pointed it out!).

@alxp alxp merged commit 75f51fb into Islandora:main Jan 19, 2023
@rosiel rosiel deleted the remote-playbook branch January 19, 2023 13:40
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.

4 participants