docker-elk icon indicating copy to clipboard operation
docker-elk copied to clipboard

To many layers

Open jonasmattsson opened this issue 8 years ago • 5 comments

Ehllo, I really like this image. It helped me a lot. But I really think you should try to optimize the dockerfile to reduce the amount of layers. You can start with chaining adjecent commands and try to COPY/ADD several files/folders in one command. read more hints on https://blog.tutum.co/2014/10/22/how-to-optimize-your-dockerfile/

jonasmattsson avatar Jun 10 '16 10:06 jonasmattsson

I haven't touched the image in a while; if so I would even move to Alpine. If you like to contribute... a PR is very welcome. :)

ChristianKniep avatar Jun 10 '16 11:06 ChristianKniep

Ok, Well it works fine for now, but's likely that I wan't an upgrade later on, then I could try to do some contrubution. Thanks for you reply. and thanks for a good job putting it together in the first place.

jonasmattsson avatar Jun 10 '16 12:06 jonasmattsson

What is your use-case? While at it I could hand-tailoe it to the needs of the users. Have you considered using a more distributed approach using multiple images?

ChristianKniep avatar Jun 10 '16 17:06 ChristianKniep

First off, I'm going to use it commersial, so i think I should be the one tailoring it and then share my changes with you. I'm creating a setup for test-environments which should mimic a bigger, full blown system, the elk container is supposed to provide the logging for developers and testers who use the plattforms, each plattform consists of 5-10 images but are not heaily used. So I'm happy with just using one image for this purpose. For the bigger system I should definitely use multiple images in order to have HA, etc. Right now i'm building my poc/pilot so i was happy to find your image wich where well documented and worked "off the shelf", so i didn't have to learn each component. Later on, when it's time to go live, I'll make some changes and probably create an image of my own (or multiple if that turns out to be a better way).

jonasmattsson avatar Jun 11 '16 07:06 jonasmattsson

See https://github.com/docker/docker/pull/22641

This was added in 1.13.0 as experimental. Might we worth trying it.

+1 for Alpine

jstarcher avatar Feb 04 '17 20:02 jstarcher