-
-
Notifications
You must be signed in to change notification settings - Fork 634
Include configuration for docker containers, SSL automation and repla… #41
Conversation
7dc2145
to
7bae57c
Compare
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.
Thanks for the PR @mofesola. Can you clarify -- did you get logstash working instead of couchdb rivers? I can test to verify as it appears that it is included in this PR, I just wanted to check if actually was working on your side.
@adeolabadmus thanks for the update. Let me know if you have any questions about how search works in HospitalRun |
@mofesola @adeolabadmus I just tried setting this up on my Mac and it fails with the following error: What OSes has this been tested on? |
@jkleinsc I have removed the erring line. Apparently, the file existed on my local so I couldn't catch it. It works fine now. Also, I reverted to v0.9.9 in server routes, because 0.9.11 was throwing an error and exiting the container. Can we look into this? Error: |
@mofesola can you change to use Node 6.x instead of 4.x? That will resolve the error you are getting with hospitalrun-server-routes v0.9.11. |
Great, I'll do that and push |
…ce CouchRiver with Logstash
This has been updated with the working logstash configuration, and node has been updated to v6 |
Looks good to me @mofesola. Thanks for the PR! I am going to merge it in. |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Replace CouchRiver with Logstash
Resolves #20