zeek / zeek-docs

Documentation for Zeek
https://docs.zeek.org
Other
50 stars 70 forks source link

install: Point to ./docker directory instead of Dockerfile #175

Closed awelzel closed 1 year ago

awelzel commented 1 year ago

Closes zeek/zeek#2805.

awelzel commented 1 year ago

@bbannier also brought up the idea of putting back the old Dockerfile, though that would mostly be an example rather than what's actually being used to create the final container images.

timwoj commented 1 year ago

@bbannier also brought up the idea of putting back the old Dockerfile, though that would mostly be an example rather than what's actually being used to create the final container images.

I think putting the Dockerfile in there would cause confusion about what it's used for.

awelzel commented 1 year ago

I've made it a bit more verbose, but essentially the same.

bbannier commented 1 year ago

@bbannier also brought up the idea of putting back the old Dockerfile, though that would mostly be an example rather than what's actually being used to create the final container images.

To be fair, I never suggested to put back dead unused code, but instead was wondering whether this would be another good reason to bring back a more simple Dockerfile (the other being that this would simplify the setup a lot where it currently requires careful interaction between Docker setups in multiple files, and a Makefile). AFAICT one reason for the current setup was the desire to mount in a local cache during docker run executing the Zeek build, but this should also be possible directly in the Dockerfile.

awelzel commented 1 year ago

Thanks @bbannier - I've opened https://github.com/zeek/zeek/issues/2810, but for now I'll move forward fixing the link.

awelzel commented 1 year ago

I've also pushed this into release/5.2.