Skip to content

Added a contrib section with a sample docker-compose file - #982

Merged
mergify[bot] merged 2 commits into
dadoonet:masterfrom
aram535:docker-compose
Jul 17, 2020
Merged

Added a contrib section with a sample docker-compose file#982
mergify[bot] merged 2 commits into
dadoonet:masterfrom
aram535:docker-compose

Conversation

@aram535

@aram535 aram535 commented Jul 16, 2020

Copy link
Copy Markdown

Fully tested complete docker-compose and Dockerfile to download and build three containers ES, KB, FSCrawler.

To enable, edit the docker-compose and change the path_to_files_to_scan to the actual path of files to be scanned. If need be, change the Dockerfile to match the version that you want to grab from github.

@dadoonet dadoonet left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great. Thanks for sharing this.
I wonder if we could add a link from the documentation to this dir?

@aram535

aram535 commented Jul 16, 2020

Copy link
Copy Markdown
Author

If you would tell me where you like to have it, I'll be happy to add a doc section as a new PR.

@dadoonet

Copy link
Copy Markdown
Owner

Ideally you can add a new commit to this PR and add some content in https://github.com/dadoonet/fscrawler/blob/master/docs/source/user/tips.rst

Thanks !

@aram535

aram535 commented Jul 17, 2020

Copy link
Copy Markdown
Author

committed.

networks:
- fscrawler_net

fscrawler:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's something that I will modify once we merge the production of the Docker image in dockerhub. #820

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah once it's in Dockerhub it'll be a lot easier. No build step needed at all for most people, some still like to build their own but most will just need the docker-compose file.

@k3ninho

k3ninho commented Aug 3, 2020

Copy link
Copy Markdown
Contributor

Doesn't work for me: the ENV lines duplicate SNAPSHOT as per... (pull request forthcoming/don't know if it's quicker to manually fix)

diff --git a/contrib/docker-compose-example/Dockerfile-fscrawler b/contrib/docker-compose-example/Dockerfile-fscrawler
index 8d02d38..33132ca 100644
--- a/contrib/docker-compose-example/Dockerfile-fscrawler
+++ b/contrib/docker-compose-example/Dockerfile-fscrawler
@@ -11,7 +11,7 @@ FROM openjdk:8-jre-alpine

ENV ES es7
ENV VERSION 2.7-SNAPSHOT
-ENV FSCRAWLER_FNAME fscrawler-$ES-$VERSION-SNAPSHOT
+ENV FSCRAWLER_FNAME fscrawler-$ES-$VERSION

RUN apk add --update
&& addgroup -S fscrawler && adduser -S -G fscrawler fscrawler

@aram535

aram535 commented Aug 3, 2020

Copy link
Copy Markdown
Author

Thanks. If you use 2.7 for VERSION, it'll work fine. this is in contrib, which means it is a template for you to copy and use. My thinking was that its for 'master' which means it'll always (it is an assumption) be a SNAPSHOT of the next version.
I do agree that it's probably a good idea to change it. Why not submit it as a pull request rather than just a diff?

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.

3 participants