Skip to content
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

Elk service to see output and dashboard metrics #4

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

machzqcq
Copy link

@machzqcq machzqcq commented Apr 1, 2017

Added elk service , so that with one command docker stack..., all services, including elasticsearch, logstash come up and just import the dashboard template and we can see the value of this repo instantly.
I opened issue-3, however was able to get around and hence submitting this pull request for you to evaluate and close the issue if this pull request is acceptable.

@bvis
Copy link
Owner

bvis commented Apr 2, 2017

Hi, this contribution is highly appreciated! Thanks for your effort. ;)

Unfortunately I saw this after trying to fix your original issue and I added my own solution.

Please, check it if you have time to let me know if it's enough for you or you think this PR could be adapted to fit with the new changes.

@machzqcq
Copy link
Author

machzqcq commented Apr 2, 2017

Thank you ! In my opinion, with my PR, we add another service to the same docker-compose yml and we are good to go (leaner over creating yet another compose file). That said, I am evaluating from my context with minimal changes. It's your repo and your decision , totally up to you ;)

machzqcq added 2 commits April 2, 2017 10:41
…nd creating new docker-compose-alternate.yml file. This is in addition to the default one that uses sebp/elk image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants