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

Add docker compose #429

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

jaywon
Copy link
Contributor

@jaywon jaywon commented Feb 24, 2019

Added simple docker-compose file for easily running the project in a Docker container with minimal setup and command line interaction.

Could add explicit docker network to run on, not sure if anyone has an opinion on that or use the default that will be created.

Copied raw Docker run commands from the wiki to README as well to have at least the base installation options on the main README and let user drive to wiki for more granular subject matter.

Hoping this makes it simple/clear for folks to start using quickly w/ a variety of options and little blockers.

@jaywon jaywon changed the title Chore/add docker compose Add docker compose Feb 24, 2019
@thc202
Copy link
Member

thc202 commented Feb 24, 2019

IMHO we should not repeat the same information in multiple places, either keep it here (probably better) or in the wiki, not both. Otherwise we have to maintain it in two places, in which case both will be out of sync/outdated very easily.

I'd suggest adding the files to a docker directory to avoid proliferation of Docker related files in the top level directory, also, one can add a more complete README there (with the information from the wiki) and just link to it from the main README.

@dscrobonia
Copy link
Contributor

I love this, @jaywon !

As per feedback, for now isolating Docker related files into a
dedicated directory.

* Move .env, .env.example, docker-compose.yml to docker/
* Update documentation for running ZAP/HUD in Docker
@jaywon
Copy link
Contributor Author

jaywon commented Mar 14, 2019

@thc202 I incorporated your feedback for moving the docker related files into it's own directory and did some copy adjustment to the main README. It feels a little strange going into a subdirectory to bring up the project but that's ok, I don't feel strongly about it. Usually this setup I would leave the docker-compose in the main directory and put all other Dockerfile and related assets into another directory and set the context appropriately but this is a bit different use case, so s/b G2G on that.

In regards to the documentation, I totally agree that info should not be duplicated but I also don't think it's intuitive or standard to have multiple README(s) in multiple places, so I didn't put a README in the docker directory for detailing instructions there. But have a few proposed solutions! 😄

In general I think developers expect to get a few things out of a main README at a minimum:

  • How do I install the project
  • How do I run the project
  • Basic configurations/usage details

Outside of that, it's standard expectation I think to go to a single wiki or branded documentation site for more descriptive information/details.

I did start this and @mellaniesori and @dscrobonia were going to jump on a call this week to really get some traction around documentation with a proposed workflow if that's ok.

Ultimately, Github Wikis really break down for really "selling" a project and organizing information well as opposed to something like this. So....long story short, I'd like to propose documentation driving towards that. Right now it will exist in wiki and the main README but not for very long. Hope that wasn't too wordy! ✏️

Copy link
Contributor

@dscrobonia dscrobonia left a comment

Choose a reason for hiding this comment

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

In general I think developers expect to get a few things out of a main README at a minimum:

How do I install the project
How do I run the project
Basic configurations/usage details

Couldn't agree more! I think its easy to overload the README as well as the wiki. I am SO excited for the new docs and I love the work/ changes to docs you've made here! Great movement towards a more refined HUD product.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants