-
Notifications
You must be signed in to change notification settings - Fork 34
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
Docker environment #134
Comments
definitely, thanks a lot for this amazing contribution! |
while still learning docker, to me it looks the only missing parts for bulk-processsor would be:
it requires to be able to launch a command provided by eg. the "bulk-processor" container from the "tomcat runtime" container. these commandlines can be overriden just by redefining them in a /ctsms/properties/ctsms-settings.properties file inside the "tomcat runtime" container. Once this works, all the features from a trial's "Job" tab become available. Think this is most important part, as this includes functionality such as eCRF data export.
|
One challenge is that I suppose to add that indirection you'd want to use some sort of task queue. I don't know what this is like in the Java world – in Python I've used celery for this. If it's too big of a change, we could put the bulk-processor into the tomcat container for now? |
i presume it should be as simple as configuring
to
but let me create a separate "docker" repo here first. |
ok, launching a container from within a container is possible in theory, but discouraged. there already is a gearman impl part of bulk-processor for job distribution/queueing, which originated form another project. but i was trying to avoid that complexity here, but just launch the job processes directly. each job reports it's state updates via rest-api.
yes i think that's the most feasible appraoch for now. |
Hello
I've been trying out Phoenix CTSMS and as part of that I have converted your
install.sh
VM-based process into a Docker environment. Everything seems to work with the exception of the Perl based bulk data loading system.If you would like to have a look:
https://github.com/adaptivehealthintelligence/docker-ctsms
If this is something you'd like to keep around, it could perhaps be turned into a PR?
The text was updated successfully, but these errors were encountered: