Fix script execution #109
Fix script execution #109
Conversation
blcham
force-pushed
the
96-script-execution
branch
from
db263a1 to
75b149f
Compare
deploy/docker-compose.yml
Outdated
| - /tmp:/tmp | ||
| - ${SHARED_ROOT:-/home}:${SHARED_ROOT:-/home} | ||
| - /usr/local/tomcat/temp/:/usr/local/tomcat/temp/ | ||
| # - /usr/local/tomcat/temp/:/usr/local/tomcat/temp/ |
There was a problem hiding this comment.
Not sure why u are commenting out this mapping. Maybe because it does not work in Windows? I guess same applies to /tmp right?
I suggest to add it as an extension .dev/dc-s-pipes-editor-rest--unify-paths-in-linux.yml, similar way as in record-manager-ui/deploy/.dev
There was a problem hiding this comment.
@blcham
I use Linux and encountered an issue with mounting the path /usr/local/tomcat/temp/. Every time I wanted to run docker-compose, I had to manually comment out
- /usr/local/tomcat/temp/:/usr/local/tomcat/temp/ . After investigating the problem, I discovered that my Docker installation was done via Snap, which caused this issue.
It might be a good idea to add a warning about this to the README. Here's a related discussion: https://stackoverflow.com/questions/45764477/docker-compose-error-while-creating-mount-source-path.
Or another solution #110 .
There was a problem hiding this comment.
This should be solved in #116
palagdan
force-pushed
the
96-script-execution
branch
from
75b149f to
aff224f
Compare
blcham
force-pushed
the
96-script-execution
branch
from
aff224f to
00c6af8
Compare
Resolve #96
I corrected the name of the repository, that is in the s-pipes-engine config.