-
Notifications
You must be signed in to change notification settings - Fork 71
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
Conversation
Current issues:
The method I used to open Fedora was the "alternatively" Here's a sample of (one node's worth of) errors in milliner log.
|
I don't see any lines marked with |
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 |
There was a problem hiding this comment.
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/ |
There was a problem hiding this comment.
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.
Is this ready for merging, do we need a final person besides Rosie to give it a final "blessing" before we merge? |
Sorry for the last minute changes - we just updated the interface for using the Playbook so I put the new info here. |
I've deleted the sneaky files that I added by mistake (thanks whoever pointed it out!). |
Purpose / why
The instructions are outdated! We no longer need karaf or so many fedora uris.
What changes were made?
example
toproduction
to give it more realism and also to match the command at the bottomVerification
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: