This repository was archived by the owner on Jul 3, 2024. It is now read-only.
DEV-667: stage item in repo & index with catalog & full-text #5
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
aelkiss
force-pushed
the
DEV-667-rebased-squashed
branch
from
43b3611 to
342292d
Compare
This attempts to reconcile earlier work here with later work in the imgsrv repo. It uses: - nginx for catalog, imgsrv fastcgi, and static files - proxy to apache for cgi It runs most containers as the current user to avoid permissions problems with cache, logs, etc. So far working: - catalog incl. CSS & JS (via shared checked-out common repo) - imgsrv fcgi & cgi - pageturner - ssd
aelkiss
force-pushed
the
DEV-667-rebased-squashed
branch
from
342292d to
51603bd
Compare
|
I think this is ready to make sure that everything works as advertised, especially on ARM. |
respinos
suggested changes
Apr 4, 2023
| depends_on: | ||
| - solr-sdr-catalog | ||
| volumes: | ||
| - "./sample-data/metadata:/app/metadata" |
There was a problem hiding this comment.
Needs - ./hathitrust_catalog_indexer:/app for the script to be found in stage_item.rb.
There was a problem hiding this comment.
But then I get
bundler: command not found: traject
Install missing gem executables with `bundle install`
It doesn't seem any bundle install persists between babel_traject_run calls.
There was a problem hiding this comment.
Argh. I think this is a side effect of using the hathitrust_catalog_indexer via the checked out copy rather than via the image. I will make the required changes...
There was a problem hiding this comment.
This should be fixed now.
With using the 'build' dockerfile instead of the image, we need to make sure that we install gems and have a place for them.
respinos
approved these changes
Apr 5, 2023
Yup, that's in the README now. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Like #1 but rebased & squashed (I didn't want to overwrite that branch since there's some other stuff happening off of it).